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

オープン 初心者向け
  • 何か問題がありますか？
#74,238 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
72/100
issue の種類
ドキュメント
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
github-actions, go, javascript, markdown
領域
documentation

調査の方向性

ドキュメントページ docs/src/content/docs/reference/safe-outputs-pull-requests.md の「Fail-Fast on Code Push Failure」セクションから始め、setup/js/safe_output_handler_manager.cjs のコードプッシュ失敗処理（1290行目付近）と比較してください。safe_output_handler_manager.test.cjs のテストが処理の継続を期待していることを確認したうえで、そのセクションと古いコメントを、テストされたランタイムの動作を説明する内容に書き直してください。ドキュメントとコメントがテスト済みのランタイム動作と一致し、コードに依存する出力をキャンセルする設定があるかどうかを回答で明らかにすれば完了です。

索引モデルが issue の本文から書いたものです。

説明

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.

主要言語
Go
スター
5.5k
フォーク
606
平均マージ
4時間 44分
マージ済み PR（30日）
799

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

github/gh-aw のほかの issue

github/gh-aw の issue をすべて見る

似ている issue

Go の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。