dev-infra: linting fails for `BUILD.bazel` file and `ng-dev format files` does not fix the failure
Los mantenedores suelen responder en 1 día
@josephperrott ya está trabajando en esto.
Desde el 3/2/2021.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
In angular/angular#40621, we run into a situation where the ng-dev format ... --check command (run as part of the lint job on CI) finds some formatting issues with a BAZEL.build file and fails. It suggests (or prompts the user) to run ng-dev format files ... to fix the issue, but that command does not fix the linting failure.
In that case, the problem was a glob pattern that included a constant string (see https://github.com/angular/angular/pull/40621#issuecomment-772482472). The fix was to manually change the pattern as follows:
-srcs = glob(["constant.string", "glob/*.pattern"]),
+srcs = ["constant.string"] + glob(["glob/*.pattern"]),
Not sure what is the best way to fix this. Ideally, we should suggest (or prompt the user) to run a command that actually fixes the error. If there is no such command, we should at least show the error message, so users can figure out how to manually fix it themselves.
- Lenguaje dominante
- JavaScript
- Estrellas
- 77
- Forks
- 83
- Merge medio
- 2 d 3 h
- PR fusionados (30 d)
- 50
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de angular/dev-infra
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
angular/dev-infra#760 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Document unpublish options for a botched releaseQuizá libre de nuevo @josephperrott la tomó hace 1531 días y no hay ningún pull request abierto. Abiertodomain: release automation
angular/dev-infra#746 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Handle RC cut failureQuizá libre de nuevo @devversion la tomó hace 1531 días y no hay ningún pull request abierto. Abiertodomain: release automation
angular/dev-infra#745 · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de angular/dev-infra
Issues similares
-
[Bug]: [A11y] Missing accessible name on icon-only standalone links (WCAG 2.1 SC 2.4.4, 4.1.2)Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
jaegertracing/jaeger-ui#4596 ·
Los mantenedores suelen responder en 1 día
-
Add: CNC Region Calima [CO]Abiertochannels:add check:passed
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
Los mantenedores suelen responder en 9 días
-
module: core
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
bigbluebutton/bigbluebutton#26028 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
twbs/bootstrap#43050 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
status:untriaged
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
midnightntwrk/servicedesk#248 ·
Los mantenedores suelen responder en 1 día