Consider adding a wrapper around `jasmine_node_test` rule
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript
- 領域
- build-system, testing
調査の方向性
まず、jasmine_node_test ルールと、issue に示されている繰り返しの bootstrap 呼び出し箇所を見つけます。提案されている ng_node_test と no_ng_node_test のエントリーポイントを、bootstrap のデフォルト値とオーバーライドのサポート付きで定義し、既存の呼び出し箇所でそれらを使用できること、また将来の ES バージョン変更で必要な編集が少なくなることを確認します。
索引モデルが issue の本文から書いたものです。
説明
PR https://github.com/angular/angular/pull/44505 updated angular/angular repo to use ES2015 instead of ES5. The mentioned change revealed that there is a duplication in a lot of places where the bootstrap is defined, for example:
jasmine_node_test(
bootstrap = ["//tools/testing:node_no_angular_es2015"],
or:
jasmine_node_test(
bootstrap = ["//tools/testing:node_es2015"],
This makes further ES version upgrades quite error-prone (and time-consuming), since we need to change ~100 files.
The proposal is to create a couple wrappers, for example:
- ng_node_test
- no_ng_node_test
and specify bootstrap there by default (and allow overrides), so that it's easier to update ES version.
- 主要言語
- JavaScript
- スター
- 77
- フォーク
- 84
- 平均マージ
- 2日 9時間
- マージ済み PR（30日）
- 42
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
angular/dev-infra のほかの issue
-
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
domain: release automation
-
domain: release automation
angular/dev-infra の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
mksglu/context-mode#1200 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
neondatabase/website#5944 ·
-
module: core
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
bigbluebutton/bigbluebutton#25849 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
jaegertracing/jaeger-ui#4506 ·