Skip to content

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

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

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

Conversation

@bourgeoa
Copy link
Copy Markdown
Contributor

Updated Node.js version from 22 to 20 in CI workflow.

Updated Node.js version from 22 to 20 in CI workflow.
Copilot AI review requested due to automatic review settings April 13, 2026 12:48
@bourgeoa bourgeoa enabled auto-merge April 13, 2026 12:48
@bourgeoa bourgeoa merged commit 9164815 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

This PR adjusts the Node.js version used in the GitHub Actions workflow, specifically in the npm publish (latest) job, to run on Node 20.

Changes:

  • Update npm-publish-latest job to use Node.js 20 (previously 22) when setting up Node before publishing.

💡 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 # required for OIDC 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 says CI is moving from Node 22 to 20, but this change only updates the npm-publish-latest job. The main build job still runs a 22/24 matrix (and the prerelease workflow input still uses 22), so CI will continue to run on Node 22/24. Either update those other Node version settings for consistency, or adjust the PR title/description to reflect that only the publish job is using Node 20.

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