Skip to content

Change Node.js version in CI workflow#177

Merged
bourgeoa merged 1 commit intomainfrom
bourgeoa-patch-2
Apr 13, 2026
Merged

Change Node.js version in CI workflow#177
bourgeoa merged 1 commit intomainfrom
bourgeoa-patch-2

Conversation

@bourgeoa
Copy link
Copy Markdown
Contributor

Updated Node.js version from 22 to 20 for compatibility.

Updated Node.js version from 22 to 20 for compatibility.
Copilot AI review requested due to automatic review settings April 13, 2026 09:24
@bourgeoa bourgeoa enabled auto-merge April 13, 2026 09:24
@bourgeoa bourgeoa merged commit 7d7f2a5 into main Apr 13, 2026
8 checks passed
Copy link
Copy Markdown
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

Adjusts the Node.js version used by the npm publishing job in the CI workflow, likely to improve compatibility when publishing to npm.

Changes:

  • Downgrade the npm-publish-latest job’s actions/setup-node version from Node 22 to Node 20.

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

Comment thread .github/workflows/ci.yml
Comment on lines 71 to 74
- uses: actions/setup-node@v6
with:
node-version: 22
node-version: 20 # for npm@latest
registry-url: 'https://registry.npmjs.org'
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 description says Node.js was updated from 22 to 20 for compatibility, but this change only affects the npm-publish-latest job; the build matrix still runs on 22 and 24 and npm-publish-dev still passes node_version: 22. Please either update the PR description to reflect the narrower scope, or align the other workflow Node versions if the intent is to downgrade CI more broadly.

Copilot uses AI. Check for mistakes.
@bourgeoa bourgeoa deleted the bourgeoa-patch-2 branch April 13, 2026 13:10
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