angular / angular/dev-infra

Consider adding a wrapper around `jasmine_node_test` rule

Offen
#338 0 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
77
Forks
83
Ø Merge
2 T. 5 Std.
Gemergte PRs (30 T.)
37

Beschreibung

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.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, die Regel jasmine_node_test und die im Issue gezeigten wiederholten Bootstrap-Aufrufstellen zu lokalisieren. Definiere die vorgeschlagenen Einstiegspunkte ng_node_test und no_ng_node_test mit Standardwerten für Bootstrap und Unterstützung für Überschreibungen. Überprüfe anschließend, dass bestehende Aufrufstellen sie verwenden können und dass zukünftige Änderungen an der ES-Version weniger Anpassungen erfordern.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
build-system, testing
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.