Open
Conversation
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Hiroshiba
commented
Feb 11, 2026
Member
Author
There was a problem hiding this comment.
わざわざファイルを追加するほどだろうかとも思ったのですが、他に良さそうなファイルがなかったので作ってみました 🙇
まあCI関連の項目足しやすいし、もし他のファイルができて統合できるってなったら統合すれば良いのかな、くらいの気持ちです 🙇
Contributor
There was a problem hiding this comment.
Pull request overview
CIワークフローの実行ポリシー(PRでは on.pull_request.paths で実行対象を絞り、main への push では網羅的に実行して乖離リスクをカバーする設計判断)を開発者向けドキュメントとして明文化するPRです。
Changes:
docs/guide/dev/ci.mdを新規作成し、CIの実行ポリシーを簡潔に記載
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # CI | ||
|
|
||
| プルリクエストでは`on.pull_request.paths`により関連するワークフローのみを実行し、CI時間を短縮する。 | ||
| pathsフィルタはコード変更に伴い実態と乖離しうるため、mainブランチへのpushでは全ワークフローを網羅的に実行することでカバーする。 |
There was a problem hiding this comment.
「mainブランチへのpushでは全ワークフローを網羅的に実行」とありますが、少なくとも audit.yml は push をトリガーにしておらず(schedule/workflow_dispatchのみ)、download_test.yml も push/pull_request がコメントアウトされています。実態としては「pathsフィルタを付けているワークフローは main への push では paths 制限なしで実行する」など、対象を限定した表現にすると正確になります。
Suggested change
| pathsフィルタはコード変更に伴い実態と乖離しうるため、mainブランチへのpushでは全ワークフローを網羅的に実行することでカバーする。 | |
| pathsフィルタはコード変更に伴い実態と乖離しうるため、pathsフィルタを付けているワークフローについては main ブランチへの push 時には paths 制限なしで実行することでカバーする。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
内容
docs/guide/dev/ci.mdを新規作成し、CIの書き方ポリシーをドキュメント化してみました。PRでは
on.pull_request.pathsによりワークフローを選択実行してCI時間を短縮しているが、pathsフィルタはコード変更に伴い実態と乖離しうる。そのためmainブランチへのpushでは全ワークフローを網羅的に実行することでカバーしているという設計判断を記載。
関連 Issue
なし
検討した他の案
8つのワークフローファイルの
paths:付近に同じコメントを追加する案。8ファイルに同じコメントを書くのは冗長なのでやめました。
既存の「プルリクエストにおけるCI」セクションに追記する案。
この情報はワークフローを編集する際に把握しておくべきポリシーであり、コントリビューター向けというよりメンテナ向けなので
docs/guide/dev/に書いてみました。その他
なし