chore: upgrade actions to Node 24 runtime (SHA-pinned) - #55
chore: upgrade actions to Node 24 runtime (SHA-pinned)#55John C. Bland II (johncblandii) wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
Updates workflow action references to SHA-pinned revisions that run on the Node 24 runtime, aligning the repo with GitHub’s Node 20 deprecation guidance and improving supply-chain safety via immutable pins.
Changes:
- Pin
actions/checkoutto a specific commit SHA with a tag comment (# v7.0.1) in affected workflows. - Pin
actions/setup-goto a specific commit SHA with a tag comment (# v7.0.0) in the build workflow. - Pin
goreleaser/goreleaser-actionto a specific commit SHA with a tag comment (# v7.2.3) in the build workflow.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/validate-codeowners.yml | Pins actions/checkout to a SHA to ensure Node 24 runtime compatibility and supply-chain safety. |
| .github/workflows/build.yml | Pins checkout, setup-go, and goreleaser-action to SHAs to avoid mutable tags and align with Node 24 runtime. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
goreleaser-action v7 installs GoReleaser v2 via 'version: latest', which rejects v1-era configs (missing version key, changelog.skip).
|
Follow-up on the flagged goreleaser concern (dae3958): |
what
SHA-pinned with precise version comments:
actions/checkout@v4→@3d3c42e5...# v7.0.1actions/setup-go@v5→@b7ad1dad...# v7.0.0goreleaser/goreleaser-action@v5→@f06c13b6...# v7.2.3why
are already being force-migrated to Node 24
matching the org's direction in chore: upgrade actions to Node 24 runtime and enforce SHA pinning .github#261
goreleaser-actioninputs in use (distribution,version,args) are unchanged in v7.Note: the workflow uses
version: latest, which already resolves GoReleaser v2.x today, while.goreleaser.ymlhas noversion: 2key — flagging as a pre-existing concern unrelated tothis pin (behavior is unchanged by this PR).
references
still on Node 20
mszostok/codeowners-validator@v0.7.1— Docker-based action (no Node runtime), not covered bythis upgrade matrix; left as-is