chore: update Node.js version and docs to match - #500
Conversation
| "vitest": "4.1.5" | ||
| }, | ||
| "engines": { | ||
| "node": "^22 || ^24" |
There was a problem hiding this comment.
Technically a breaking change for downstream users
There was a problem hiding this comment.
Should we keep Node.js 22 support and testing but specify devEngines to make the supported versions for development more explicit?
There was a problem hiding this comment.
I originally removed devEngines in #397 because the package manager settings there broke dependabot, but we could likely add the runtime portion back here to cover this intent?
There was a problem hiding this comment.
While a breaking changing, we have no way to version this appropriate yet until it hits 1.0... so what do we want to do here?
There was a problem hiding this comment.
Is there a strong reason to drop Node.js 22 support as part of the version alignment here?
There was a problem hiding this comment.
I was concerned with the potential incompatibilities between npm 10.x included with Node 22.x and npm 11.x included with Node 24.x.
There was a problem hiding this comment.
That should only impact the Node.js version used for the project, I believe. As long as we restrict the dev version we should be good 🙏🏻
3 participants
Pull Request
Updates the version of Node.js to latest 24.20.0 release. Also did doc updates to only allow for Node.js 24.x due to package manager differences and more reflective of the version in .nvmrc.
Proposed Changes
Updating to specify only Node 24.x
Readiness Checklist
Author/Contributor
npm run formatand fix any formatting issues that have been introducednpm run lintand fix any linting issues that have been introducednpm run testand run tests