ci(analysis): bump test runner node_version to 24 - #2840
Merged
Merged
Conversation
Aligns the Backend Tests and Frontend Tests action runners with the Node 24 runtime images used in Dockerfiles and Renovate lockfile maintenance.
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The focused CI configuration update has no unresolved issues.
Pull request overview
Aligns CI test runners with the Node 24 production and lockfile environments.
Changes:
- Updates backend and frontend analysis jobs from Node 22 to Node 24.
- Prevents npm 10/11 lockfile compatibility failures.
File summaries
| File | Description |
|---|---|
.github/workflows/analysis.yml |
Runs backend and frontend tests with Node 24. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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
Bump the
node_versioninput from"22"to"24"forbackend-testsandfrontend-testsin.github/workflows/analysis.yml.Context
backend/Dockerfileandfrontend/Dockerfile) and@types/nodedependencies are already on Node 24.package-lock.jsonusing Node 24 / npm 11. When CI runs on Node 22 (npm 10),npm cifails withcode EUSAGEdue to npm 11 pruning optional nested peer dependencies that npm 10 expects.Upstream proposal to auto-detect Node version in
bcgov/actions/test-and-analyse: bcgov/actions#183Thanks for the PR!
Deployments, as required, will be available below:
Please create PRs in draft mode. Mark as ready to enable:
After merge, new images are deployed in: