Consider adding a wrapper around `jasmine_node_test` rule
- Langage dominant
- JavaScript
- Étoiles
- 77
- Forks
- 83
- Merge moyen
- 2 j 5 h
- PR mergées (30 j)
- 37
Description
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.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
Commencez par localiser la règle jasmine_node_test et les sites d’appel répétés de bootstrap présentés dans l’issue. Définissez les points d’entrée proposés ng_node_test et no_ng_node_test avec des valeurs de bootstrap par défaut et la prise en charge des remplacements, puis vérifiez que les sites d’appel existants peuvent les utiliser et que les futurs changements de version d’ES nécessitent moins de modifications.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- build-system, testing
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100