nodejs / nodejs/node-core-utils
Cancelled GitHub CI rerun after success is treated as failure
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 314
- Forks
- 137
- Merge medio
- 2 d 8 h
- PR fusionados (30 d)
- 24
Descripción
PR: https://github.com/nodejs/node/pull/43085
Action rerun attempt: https://github.com/nodejs/node/actions/runs/2320332585/attempts/7
git-node land output: https://github.com/nodejs/node/pull/43085#issuecomment-1126858386
At the moment of landing, the latest run of that action was cancelled, there were 5 prior successful runs, and GitHub UI reported everything as green.
However, ncu treated it as Last GitHub CI failed error, which is either wrong (not sure if easily fixable, as it requires failproof checking previous attempts) or confusing (Last GitHub CI cancelled might be a better error message).
Details: in PRChecker.checkGitHubCI(), commit.checkSuites.nodes[2] for this case was
{
app: { slug: 'github-actions' },
conclusion: 'CANCELLED',
status: 'COMPLETED'
}
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.
Línea de trabajo
Comienza en PRChecker.checkGitHubCI() e inspecciona cómo se interpreta commit.checkSuites.nodes, usando el PR vinculado, el intento de la acción y la salida de git-node land como contexto de reproducción. Verifica el comportamiento cuando la ejecución más reciente de GitHub Actions se cancela después de ejecuciones anteriores exitosas. Se considera terminado cuando el estado de CI informado y el mensaje de error reflejan correctamente ese caso, con pruebas que lo cubran.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- cli
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 55/100