Repository navigation
ci(ENG-3243): move action pins off the deprecated Node 20 runtime - #51
Merged
Merged
Conversation
Swap every JavaScript action that still declares the node20 (or older) runtime for a release that declares node24, keeping this repo's pin style. Workflows that push take checkout v5 rather than v6, so their credential handling is unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
🟢 Approval recommended
Both SHAs match the stated releases, use Node.js 24, and preserve the required workflow inputs.
0 open findings
What changed in this PR
Updates workflow actions from deprecated Node.js 20 runtimes to Node.js 24.
Changes:
- Upgrades
actions/checkoutto v6.0.2. - Upgrades
release-please-actionto v5.0.0. - Retains immutable SHA pinning.
| File | Description |
|---|---|
.github/workflows/validate.yml |
Updates checkout to Node.js 24-compatible v6.0.2. |
.github/workflows/release-please.yml |
Updates release-please to Node.js 24-compatible v5.0.0. |
🧠 Review effort: Balanced
Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
No issues found across 2 files
Auto-approved: Bumps the two GitHub Actions pins (checkout v4→v6, release-please v4→v5) to the Node 24 runtime; a workflow-only, bounded tooling update that clears deprecation annotations and is easily reversible.
View guided diff | Turn on auto-fix | Re-trigger cubic
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.
Summary
Moves every JavaScript action in this repo's workflows that declares the Node 20 (or older) runtime to a release that runs on Node 24, removing the "Node.js 20 is deprecated" annotations GitHub adds to every job that uses one. Part of ENG-3243, which has done the same across the org.
actions/checkoutde0fac2)googleapis/release-please-action45996ed)Compatibility
$RUNNER_TEMPinstead of.git/config;gitcommands keep working. No workflow here pushes or runs in a container job.Testing
Workflow-only change. Steps that only run on push, release, deploy or schedule aren't exercised by this PR's CI and will first run on merge.
🤖 Generated with Claude Code
Summary by cubic
Moves this repo's two JavaScript workflow actions off the deprecated Node 20 runtime, removing the "Node.js 20 is deprecated" annotations GitHub adds to every job that uses them.
actions/checkoutbumps v4 → v6.0.2. The only change that matters here is that persisted credentials now live under$RUNNER_TEMPinstead of.git/config; no workflow pushes in a way that depends on the old location.googleapis/release-please-actionbumps v4.4.1 → v5.0.0, which only changes the runtime.Workflow-only change: steps that only run on push, release, deploy, or schedule aren't exercised by this PR's CI and will first run on merge.
Written for commit 37a4669. Summary will update on new commits.