ci: merge ジョブ名の matrix 参照をやめて未展開表示を解消 - #7031
Conversation
- job レベルの if が false の場合、 matrix は展開されず checks 一覧に
`merge ${{ matrix.php }}` がそのまま表示される
- name が matrix を参照しなければ GitHub が matrix 値を後置するため、
展開時の表示は `merge (8.2)` となり従来と同じ情報量を保てる
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthrough
ChangesGitHub Actions ワークフロー
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 4.4 #7031 +/- ##
==========================================
+ Coverage 77.17% 77.20% +0.03%
==========================================
Files 564 564
Lines 28082 28082
==========================================
+ Hits 21671 21681 +10
+ Misses 6411 6401 -10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
1 participant
概要(Overview・Refs Issue)
PR の checks 一覧に
dockerbuild / merge ${{ matrix.php }}が未展開のまま表示される問題を解消します。dockerbuild.ymlのmergeジョブはif: github.event_name != 'pull_request'を持つため、PR では job レベルで skip されます。job レベルの
ifが false のときは matrix が展開されないので、name:に書いた${{ matrix.php }}が置換されず、テンプレート文字列のまま表示されます。実際の表示例です（該当ジョブは 4 件ではなく 1 件・skipped）。
方針(Policy)
name:から matrix 参照を外しました。name:が matrix を参照していない場合、GitHub は matrix の値を括弧で後置します。本リポジトリ内の既存ジョブがそのまま実例です。
name: dockerbuild（dockerbuild.yml）dockerbuild (linux/amd64, 8.3, pgsql)name: PHPUnit（unit-test.yml）PHPUnit (ubuntu-24.04, 8.2, pgsql13)このため
name: mergeとしても、展開されるとき（push / tag）の表示はmerge (8.2)〜merge (8.5)となり、従来のmerge 8.2と情報量は変わりません。skip されるときは name に式が含まれないため、テンプレート文字列は表示されません。
実装に関する補足(Appendix)
name:の 1 行とその意図を残すコメントのみsuccessのみのため、ジョブ名の変更による branch protection への影響はなしテスト（Test)
actionlintを実行し、本変更による新規指摘がないことを確認（既存の SC2086 info 3 件のみで、変更前後で同一）mergeが 1 件・skipped）merge (8.2)〜）はマージ後の push 実行で確認できます相談（Discussion）
e2e-test.ymlのplaywrightジョブも同じ構造で、workflow_dispatch実行時にPlaywright (${{ matrix.suite }})が未展開で表示されます（fork で workflow_dispatch を回して確認済みです）。ただしこちらは matrix が php / db / suite の 3 軸あり、同じ対処をすると 22 件の check 名が
Playwright (8.5, pgsql, admin-top)の形に変わります。表示が長くなる割に効果が手動実行時に限られるため本 PR には含めていませんが、揃えた方がよければ別 PR で対応いたします。
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目
Summary by CodeRabbit