Consider adding a wrapper around `jasmine_node_test` rule
- Lingua principale
- JavaScript
- Stelle
- 77
- Fork
- 83
- Merge medio
- 2g 5h
- PR unite (30g)
- 37
Descrizione
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.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia individuando la regola jasmine_node_test e i siti di chiamata ripetuti di bootstrap mostrati nell’issue. Definisci gli entry point proposti ng_node_test e no_ng_node_test con valori predefiniti di bootstrap e supporto per gli override, quindi verifica che i siti di chiamata esistenti possano usarli e che le future modifiche alla versione di ES richiedano meno interventi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript
- Ambito
- build-system, testing
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100