Skip to content

ci: move checkout and cache actions to Node 24 releases - #814

Merged
behinddwalls merged 2 commits into
mainfrom
preetam/ci-node24
Oct 8, 2026
Merged

behinddwalls merged 2 commits into
mainfrom
preetam/ci-node24

Conversation

@behinddwalls

@behinddwalls behinddwalls commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Why?

Every workflow run warns that actions/checkout@v4.3.1 and actions/cache@v4 target the deprecated Node.js 20 runtime and are being forced onto Node 24 (e.g. https://github.com/uber/submitqueue/actions/runs/37717545209).

What?

  • actions/checkout v4.3.1 → v7.0.1 in ci.yml, docs.yml and tag-go-pseudoversion.yml.
  • actions/cache v4 → v6.1.0 in .github/actions/setup.
  • Both stay SHA-pinned. No breaking change applies here: jobs run on GitHub-hosted runners (minimum runner version met), checkout v6+ still persists credentials for tag-go-pseudoversion.yml's tag push (now in a separate file), and v7's pull_request_target fork block does not affect this repo.

Test Plan

✅ zizmor 1.25.2 (repo config) and actionlint clean on .github/
✅ Every directly pinned action now declares node24 (the rest are composite)
Post-merge: confirm the Node.js 20 annotation is gone from the next main CI and Docs runs, and that tag-go-pseudoversion still pushes its tag.

🤖 Generated with Claude Code

## Summary

### Why?

Every workflow run warns that `actions/checkout@v4.3.1` and `actions/cache@v4` target the deprecated Node.js 20 runtime and are being forced onto Node 24 (e.g. https://github.com/uber/submitqueue/actions/runs/37717545209).

### What?

- `actions/checkout` v4.3.1 → v7.0.1 in `ci.yml`, `docs.yml` and `tag-go-pseudoversion.yml`.
- `actions/cache` v4 → v6.1.0 in `.github/actions/setup`.
- Both stay SHA-pinned. No breaking change applies here: jobs run on GitHub-hosted runners (minimum runner version met), checkout v6+ still persists credentials for `tag-go-pseudoversion.yml`'s tag push (now in a separate file), and v7's `pull_request_target` fork block does not affect this repo.

## Test Plan

✅ zizmor 1.25.2 (repo config) and actionlint clean on `.github/`
✅ Every directly pinned action now declares `node24` (the rest are composite)
Post-merge: confirm the Node.js 20 annotation is gone from the next `main` CI and Docs runs, and that `tag-go-pseudoversion` still pushes its tag.
@behinddwalls
behinddwalls marked this pull request as ready for review October 8, 2026 02:30
@behinddwalls
behinddwalls requested a review from a team as a code owner October 8, 2026 02:30
@behinddwalls
behinddwalls merged commit dcd2807 into main Oct 8, 2026
18 checks passed
@behinddwalls
behinddwalls deleted the preetam/ci-node24 branch October 8, 2026 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant