angular / angular/dev-infra

Consider adding a wrapper around `jasmine_node_test` rule

Aperta
#338 0 commenti 2 reazioni 0 assegnatari Vedi su GitHub
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.