Safe-output docs still promise fail-fast cancellation after it was removed

Abierto Apto para principiantes
  • ¿Algo no está bien?
#74,238 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
72/100
Tipo de issue
Documentación
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
github-actions, go, javascript, markdown
Área
documentation

Línea de trabajo

Empieza por la página de documentación docs/src/content/docs/reference/safe-outputs-pull-requests.md, sección "Fail-Fast on Code Push Failure", y compárala con setup/js/safe_output_handler_manager.cjs alrededor del manejo de fallos de envío de código (cerca de la línea 1290). Confirma que las pruebas en safe_output_handler_manager.test.cjs esperan que el procesamiento continúe y, después, reescribe la sección de documentación y los comentarios obsoletos para describir ese comportamiento. Listo cuando la documentación y los comentarios coincidan con el comportamiento en tiempo de ejecución probado, y la respuesta indique si alguna configuración cancela las salidas que dependen del código.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

The “Fail-Fast on Code Push Failure” docs state:

If push-to-pull-request-branch (or create-pull-request) fails, the safe-output pipeline cancels all remaining non-code-push outputs.

However, PR #33295 removed that cancellation guard to address issue #33285, where a failed PR creation cancelled unrelated outputs.

The implementation and tests now agree that processing continues:

  • The v0.93.5 runtime records code-push failures and continues with subsequent outputs. Later add_comment outputs receive a failure warning.
  • The tests explicitly expect comments and issue creation to continue after a failed push.
  • The docs and some runtime comments still describe cancellation. The mismatch remains on current main.

Could you please update the docs and stale code comments to describe the current behavior? Please also clarify whether a supported configuration exists for cancelling outputs that depend on successful code delivery.

This report concerns documentation accuracy. The evidence comes from source and test inspection; no runtime behavior change is requested.

Lenguaje dominante
Go
Estrellas
5.5k
Forks
606
Merge medio
4 h 44 min
PR fusionados (30 d)
799

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de github/gh-aw

Todos los issues de github/gh-aw

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.