Consider adding a wrapper around `jasmine_node_test` rule
- Lenguaje dominante
- JavaScript
- Estrellas
- 77
- Forks
- 83
- Merge medio
- 2 d 5 h
- PR fusionados (30 d)
- 37
Descripción
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Línea de trabajo
Comienza localizando la regla jasmine_node_test y los puntos de llamada repetidos de bootstrap mostrados en el issue. Define los puntos de entrada propuestos ng_node_test y no_ng_node_test con valores predeterminados de bootstrap y compatibilidad con sobrescrituras; después, verifica que los puntos de llamada existentes puedan usarlos y que los futuros cambios de versión de ES requieran menos modificaciones.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- build-system, testing
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100