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

Aperta Adatta ai principianti
  • Qualcosa non va?
#74,238 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
72/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
github-actions, go, javascript, markdown
Ambito
documentation

Direzione di ricerca

Inizia dalla pagina di documentazione docs/src/content/docs/reference/safe-outputs-pull-requests.md, sezione "Fail-Fast on Code Push Failure", e confrontala con setup/js/safe_output_handler_manager.cjs intorno alla gestione dei fallimenti del push del codice (vicino alla riga 1290). Conferma che i test in safe_output_handler_manager.test.cjs si aspettano che l'elaborazione continui, poi riscrivi la sezione della documentazione e i commenti obsoleti per descrivere quel comportamento. Completato quando documentazione e commenti corrispondono al comportamento testato a runtime e la risposta indica se una configurazione annulla gli output che dipendono dal codice.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
Go
Stelle
5.5k
Fork
606
Merge medio
4h 44m
PR unite (30g)
799

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di github/gh-aw

Tutte le issue di github/gh-aw

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.