Conversation
Updated Node.js version from 22 to 20 for compatibility.
There was a problem hiding this comment.
Pull request overview
This PR updates the Node.js version used in the CI workflow’s npm publishing job, aiming to improve compatibility for the publish step.
Changes:
- Pin
npm-publish-latestjob to Node.js 20 duringactions/setup-node.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - uses: actions/setup-node@v6 | ||
| with: | ||
| node-version: 22 | ||
| node-version: 20 # required for OIDC npm@latest | ||
| registry-url: 'https://registry.npmjs.org' |
There was a problem hiding this comment.
PR title/description say CI is moving from Node 22 to 20 for compatibility, but only the npm-publish-latest job is pinned to Node 20. The build/test matrix still runs on 22 and 24 (and the artifact used for publishing is produced under Node 22), so Node 20 compatibility isn’t validated. Either add Node 20 to the build matrix / align the artifact build version with the publish job, or update the PR title/description to reflect that this change is only for the publish step.
Updated Node.js version from 22 to 20 for compatibility.