Upodate checkout GHA step across all workflows#3661
Conversation
We use different versions across the folows. Some version are no longer supported by latest runners. Update all to consistently use the immutable latest version. (cherry picked from commit c713a03)
PR SummaryLow Risk Overview Each updated step keeps existing Reviewed by Cursor Bugbot for commit 563ce80. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
There was a problem hiding this comment.
LGTM — mechanical cherry-pick standardizing actions/checkout across all workflows to a single immutable v7.0.0 commit SHA.
Extended reasoning...
Overview
This PR is a cherry-pick of already-merged #3659. It updates 17 GitHub Actions workflow YAML files to consistently use actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 (the immutable commit SHA for v7.0.0), replacing a mix of @v2 through @v6 tags. All existing with: options (submodules, fetch-depth, ref, etc.) are preserved verbatim, and a comment is added pointing at the v7.0.0 release. No application code is touched.
Security risks
None — in fact this is a security improvement. Pinning to an immutable commit SHA is the recommended supply-chain hardening practice for third-party GitHub Actions, since floating tags can be retagged by the action author. The chosen action (actions/checkout) is an official GitHub-maintained action, so this is uncontroversial.
Level of scrutiny
Very low. CI infrastructure config only, no production code paths, no logic changes, and the diff is fully mechanical (same SHA pasted in every spot, same options preserved). The original PR #3659 was already reviewed and merged, so this is just propagating the same change.
Other factors
The cursor[bot] PR Summary also flagged this as low risk and confirms the option-preservation observation. The bug hunting system found no issues. The only caveat is that older actions/checkout versions may bring slightly different default behavior (e.g., fetch-depth defaults, ref handling), but the PR description explicitly motivates the change as needed for runner compatibility, and any post-merge CI breakage would surface immediately on the next workflow run.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## release/v6.6 #3661 +/- ##
================================================
- Coverage 58.89% 58.01% -0.89%
================================================
Files 2225 2151 -74
Lines 183480 174933 -8547
================================================
- Hits 108068 101492 -6576
+ Misses 65704 64433 -1271
+ Partials 9708 9008 -700
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
We use different versions across the folows. Some version are no longer supported by latest runners.
Update all to consistently use the immutable latest version.