angular / angular/dev-infra

Consider adding a wrapper around `jasmine_node_test` rule

オープン
#338 コメント 0 件 リアクション 2 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
77
フォーク
83
平均マージ
2日 5時間
マージ済み PR(30日)
37

説明

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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

まず、jasmine_node_test ルールと、issue に示されている繰り返しの bootstrap 呼び出し箇所を見つけます。提案されている ng_node_test と no_ng_node_test のエントリーポイントを、bootstrap のデフォルト値とオーバーライドのサポート付きで定義し、既存の呼び出し箇所でそれらを使用できること、また将来の ES バージョン変更で必要な編集が少なくなることを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript
領域
build-system, testing
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。