Skip to content

enforce node version (>=22) - for main#1250

Merged
ksen0 merged 3 commits intoprocessing:mainfrom
nbogie:nb-enforce-node-version-main
Mar 18, 2026
Merged

enforce node version (>=22) - for main#1250
ksen0 merged 3 commits intoprocessing:mainfrom
nbogie:nb-enforce-node-version-main

Conversation

@nbogie
Copy link
Contributor

@nbogie nbogie commented Mar 17, 2026

Addresses #1233 for main branch

changes

  • package.json and .npmrc now stipulate and enforce a node.js version >= 22
    (Trying to run npm install or npm ci will fail on older versions of node.)
  • CI: In test.yml specifies for all github actions to use v22
  • CI: Tells astro build to use node v22 in build-deploy job (astro does its own npm install which will otherwise fail)
  • docs: update technical_overview doc to specify node version >= 22 as a pre-req.

Why?

The codebase now includes code that won't run on earlier node versions, and support for node v20 ends in April 2026.
https://nodejs.org/en/about/previous-releases
Read more in #1233

what about 2.0 branch?

For 2.0 branch this work was covered in two PRs: #1234 and #1241

ci: have github test.yml actions to use node v22
ci: deploy: tell astro build to use node version 22
package+npmrc: enforce node version >= 22
Stipulate node v22 in setup in technical_overview
@nbogie nbogie force-pushed the nb-enforce-node-version-main branch from 31e0aa6 to a7dd810 Compare March 17, 2026 16:13
@nbogie
Copy link
Contributor Author

nbogie commented Mar 17, 2026

CI tested ok, including build-deploy to github-pages.

@nbogie nbogie marked this pull request as ready for review March 17, 2026 17:45
@nbogie
Copy link
Contributor Author

nbogie commented Mar 17, 2026

(I'm happy for this to be merged)

@ksen0 ksen0 merged commit ac8e822 into processing:main Mar 18, 2026
4 checks passed
@nbogie nbogie deleted the nb-enforce-node-version-main branch March 18, 2026 10:59
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.

2 participants