fix(ci-cd): bump github actions to node 24 runtimes - #281
Open
piyushsinghgaur1 wants to merge 2 commits into
Open
fix(ci-cd): bump github actions to node 24 runtimes#281piyushsinghgaur1 wants to merge 2 commits into
piyushsinghgaur1 wants to merge 2 commits into
Conversation
- bump actions/checkout from v4 to v5 in main.yaml, release.yaml, sync-docs.yaml (both checkout steps) and trivy.yaml - bump actions/setup-node from v4 to v5 in main.yaml and release.yaml - set release.yaml node-version from '22' to '24' - drop the stale "UPDATED to v4" trailing comment in release.yaml - keep the main.yaml CI matrix at [22, 24] so node 22 coverage is retained - leave the SHA-pinned aquasecurity/trivy-action untouched clears the "Node.js 20 is deprecated" GitHub Actions annotation, since checkout@v4 and setup-node@v4 declare using: node20 while the v5 majors declare using: node24. GH-280
Regenerate package-lock.json from scratch and apply npm audit fix to clear vulnerabilities reported by the Trivy scan. GH-280
|
piyushsinghgaur1
marked this pull request as ready for review
August 18, 2026 08:53
Sourav-kashyap
approved these changes
Aug 18, 2026
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.



Description
GitHub Actions runs in this repository were emitting the annotation:
actions/checkout@v4andactions/setup-node@v4declareusing: node20in theiraction.yml, so the runner executed them on the deprecated Node 20 action runtime. Thev5majors of both declareusing: node24. This PR bumps those pins and moves the release job onto Node 24.Each target major was verified to actually be on node24 before bumping:
Action pin changes
.github/workflows/main.yamlactions/checkout@v4@v5.github/workflows/main.yamlactions/setup-node@v4@v5.github/workflows/release.yamlactions/checkout@v4@v5.github/workflows/release.yamlactions/setup-node@v4@v5.github/workflows/sync-docs.yamlactions/checkout(Checkout Extension Code)@v4@v5.github/workflows/sync-docs.yamlactions/checkout(Checkout Docs Repository)@v4@v5.github/workflows/trivy.yamlactions/checkout@v4@v5Node version changes
.github/workflows/release.yamlnode-version(Setup Node)'22''24'.github/workflows/main.yamlnode-versionmatrix[22, 24][22, 24](unchanged)Also removed the now-inaccurate trailing
# UPDATED to v4comment on thesetup-nodestep inrelease.yaml.Deliberately left unchanged
main.yamlCI matrixnode-version: [22, 24]— it already includes Node 24, so it does not cause the annotation. Dropping 22 would reduce test coverage for consumers still on Node 22, so the matrix is untouched.aquasecurity/trivy-action@57a97c7e7821a5776cebc9bb87c984fa69cba8f1— third-party action pinned by commit SHA for supply-chain reasons. Re-pointing it is out of scope for this change and it is not what the Node 20 annotation refers to.actions/github-script,actions/setup-python,actions/upload-artifactoractions/cacheusages exist in this repo, so there was nothing else to bump and no legacy Octokit:owner/:reporoute placeholders to migrate.Fixes #280
Type of change
How Has This Been Tested?
using: node24via the GitHub contents API (output above), so the deprecation annotation is actually resolved rather than just moved.js-yamlload ofmain.yaml,release.yaml,sync-docs.yaml,trivy.yaml— all OK).release.yaml/sync-docs.yamlonly trigger onworkflow_dispatch/ push tomasterrespectively, so they will be exercised after merge).Checklist:
npm run teston Node 22 and 24.