Skip to content

Change Node.js version to 20 in CI workflow#183

Merged
bourgeoa merged 1 commit intomainfrom
bourgeoa-npm
Apr 13, 2026
Merged

Change Node.js version to 20 in CI workflow#183
bourgeoa merged 1 commit intomainfrom
bourgeoa-npm

Conversation

@bourgeoa
Copy link
Copy Markdown
Contributor

Updated Node.js version from 22 to 20 for compatibility with npm@latest.

Updated Node.js version from 22 to 20 for compatibility with npm@latest.
Copilot AI review requested due to automatic review settings April 13, 2026 09:17
@bourgeoa bourgeoa enabled auto-merge April 13, 2026 09:18
@bourgeoa bourgeoa merged commit b720c1e into main Apr 13, 2026
8 checks passed
@bourgeoa bourgeoa deleted the bourgeoa-npm branch April 13, 2026 09:18
Copy link
Copy Markdown

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

This PR updates the Node.js version used in the npm-publish-latest GitHub Actions job, aiming to use Node 20 (not 22) due to npm@latest compatibility concerns during publishing.

Changes:

  • Changed actions/setup-node version in the npm-publish-latest job from Node 22 to Node 20.

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

Comment on lines 78 to 82
- uses: actions/setup-node@v6
with:
node-version: 22
node-version: 20 # for npm@latest
registry-url: 'https://registry.npmjs.org'
- name: Update npm to latest (required for OIDC)
Copy link

Copilot AI Apr 13, 2026

Choose a reason for hiding this comment

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

The PR title/description suggest the CI workflow is moving from Node 22 to Node 20, but this change only updates the Node version for the npm-publish-latest job. The build job matrix still runs on 22/24 and the publish artifact is still produced only when matrix.node-version == 22 (and npm-publish-dev still uses 22). Either update the workflow consistently (matrix + artifact condition + publish-dev) or adjust the PR title/description to clarify that only the release publishing job is pinned to Node 20.

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.

2 participants