expected_assets checks should validate HTML docs were generated
Los mantenedores suelen responder en 3 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 57/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- javascript
- Área
- release
Línea de trabajo
Read ansible/www-standalone/tools/promote/expected_assets/v26.x and ansible/www-standalone/tools/promote/check_assets.js, especially the deliberate top-level-only check noted around lines 57–58. Start by tracing how expected asset lists are consumed during release promotion and how the Node.js documentation HTML is generated. Done means promotion checks detect missing generated HTML docs; no test or exact checking strategy is named in the issue.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
See https://github.com/nodejs/node/issues/66571.
Presently our expected_assets checks during release promotion are only checking a few things under docs, e.g. for v26.x:
https://github.com/nodejs/build/blob/0cebb1079f998ac407fb882e144b8e377f23a105/ansible/www-standalone/tools/promote/expected_assets/v26.x#L1-L5
So we did not catch that no HTML files were generated.
I don't know the historical reason for only checking the top level docs (speculating here, but maybe time and/or the annoyance of listing out all 200+ files in each expected asset list?), but it does appear to be deliberate, e.g. https://github.com/nodejs/build/blob/0cebb1079f998ac407fb882e144b8e377f23a105/ansible/www-standalone/tools/promote/check_assets.js#L57-L58
- Lenguaje dominante
- Jinja
- Estrellas
- 542
- Forks
- 184
- Merge medio
- 3 d 5 h
- PR fusionados (30 d)
- 10
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
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 nodejs/build
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
nodejs/build#4514 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
nodejs/build#4482 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
nodejs/build#4324 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 52/100
nodejs/build#4513 · 1 reacción ·
Los mantenedores suelen responder en 3 días
-
Windows/arm machines unavailablePosiblemente ocupada @StefanStojanovic la tomó hace 3 días. Abiertoincident platform:windows
nodejs/build#4511 · 1 comentario · 1 reacción · 1 asignado ·
Los mantenedores suelen responder en 3 días
Todos los issues de nodejs/build
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
adidshaft/grok-gadgets-linux-sdk#70 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
-
RPBCAbiertoawaiting policy acceptance precheck-passed
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
Bioconductor/BiocContributions#240 · 1 comentario ·
-
[Package Request]: radare.iaitoAbiertoHelp-Wanted Needs-Triage Package-Request
Dificultad 2/5 1-3 horas Aptitud para principiantes 71/100
microsoft/winget-pkgs#451272 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Add new release repositoriesAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 77/100
ros2-gbp/ros2-gbp-github-org#1167 ·
Los mantenedores suelen responder en 1 día