angular / angular/dev-infra

Consider adding a wrapper around `jasmine_node_test` rule

Abierto
#338 0 comentarios 2 reacciones 0 asignados Ver en GitHub
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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.