Skip to content

[WIP] Upgrade project to support Node.js 22 and 24#939

Merged
decyjphr merged 2 commits intomain-enterprisefrom
copilot/upgrade-nodejs-to-22-and-24
Mar 2, 2026
Merged

[WIP] Upgrade project to support Node.js 22 and 24#939
decyjphr merged 2 commits intomain-enterprisefrom
copilot/upgrade-nodejs-to-22-and-24

Conversation

Copy link
Contributor

Copilot AI commented Mar 2, 2026

  • Identify all Node.js version references
  • Update node-ci.yml matrix from 18, 2022, 24
  • Update .nvmrc from v20v22
  • Update package.json engines from >= 16.0.0>= 22.0.0
  • Update Dockerfile from node:20-alpinenode:22-alpine
  • Update rc-release.yml node-version from 16.x22.x
  • Update create-release.yml node-version from 16.x22.x
  • Update create-pre-release.yml node-version from 16.x22.x

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: decyjphr <57544838+decyjphr@users.noreply.github.com>
@decyjphr decyjphr marked this pull request as ready for review March 2, 2026 22:52
Copilot AI review requested due to automatic review settings March 2, 2026 22:52
@decyjphr decyjphr merged commit f22d6a7 into main-enterprise Mar 2, 2026
8 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates this repository’s declared and CI/test Node.js versions to align with a move to Node.js 22 (and CI coverage for 24), spanning local dev, Docker builds, and GitHub Actions workflows.

Changes:

  • Bumped package.json engines.node to >= 22.0.0.
  • Updated CI workflow matrix to test on Node 22 and 24, and updated release workflows to use Node 22.x.
  • Updated runtime/tooling pins for local/dev and container builds (.nvmrcv22, Dockerfilenode:22-alpine).

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
package.json Raises minimum supported Node version via engines.node.
Dockerfile Updates container base image to Node 22 Alpine.
.nvmrc Aligns local nvm default to Node 22.
.github/workflows/rc-release.yml Uses Node 22.x for the RC release workflow.
.github/workflows/node-ci.yml Updates CI matrix to run on Node 22 and 24.
.github/workflows/create-release.yml Uses Node 22.x for the release workflow.
.github/workflows/create-pre-release.yml Uses Node 22.x for the pre-release workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

},
"engines": {
"node": ">= 16.0.0"
"node": ">= 22.0.0"
Copy link

Copilot AI Mar 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR description/checklist says all Node.js version references were identified/updated, but there are still hard-coded Node 16 runtime/version references elsewhere (e.g. .github/actions/publish-docker/action.yml uses node-version: 16.x, and template.yml uses Runtime: nodejs16.x; serverless.yml still pins nodejs20.x). Either update those references as part of this PR or adjust the PR scope/description so it’s accurate.

Copilot uses AI. Check for mistakes.
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.

3 participants