Skip to content

ci: test on node.js 24 #6778

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

ci: test on node.js 24 #6778

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/benchmark.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
- name: Setup Node.js environment
uses: actions/setup-node@v6
with:
node-version: '22'
node-version: '24'
cache: 'npm'
check-latest: true

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pre-release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
fetch-depth: 0
- uses: actions/setup-node@v6
with:
node-version: '22'
node-version: '24'
check-latest: true
registry-url: 'https://registry.npmjs.org'
# Use npm@10 on Node 22+ due to https://github.com/npm/cli/issues/8489
Expand Down
12 changes: 6 additions & 6 deletions .github/workflows/workflow.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,12 @@ jobs:
if: ${{ !steps.release-check.outputs.IS_RELEASE }}
- uses: actions/setup-node@v6
with:
node-version: 22
node-version: '24'
if: ${{ !steps.release-check.outputs.IS_RELEASE }}
# Use npm@10 on Node 22+ due to https://github.com/npm/cli/issues/8489
- name: Setup npm version
run: npm install -g npm@10
if: ${{ !steps.release-check.outputs.IS_RELEASE && matrix.node-version == '22' }}
if: ${{ !steps.release-check.outputs.IS_RELEASE }}
- name: Install dependencies
run: npm ci
if: ${{ !steps.release-check.outputs.IS_RELEASE }}
Expand All @@ -52,7 +52,7 @@ jobs:
strategy:
matrix:
os: [ubuntu-24.04, macos-14, windows-2025]
node-version: ['22']
node-version: ['24']
# Must include the minimum deno version from the `DENO_VERSION_RANGE` constant in `node/bridge.ts`.
deno-version: ['v2.4.2']
include:
Expand Down Expand Up @@ -83,7 +83,7 @@ jobs:
# Use npm@10 on Node 22+ due to https://github.com/npm/cli/issues/8489
- name: Setup npm version
run: npm install -g npm@10
if: ${{ !steps.release-check.outputs.IS_RELEASE && matrix.node-version == '22' }}
if: ${{ !steps.release-check.outputs.IS_RELEASE && matrix.node-version != '18.14.0' }}
- name: Setup Deno
uses: denoland/setup-deno@v1
with:
Expand Down Expand Up @@ -113,7 +113,7 @@ jobs:
strategy:
matrix:
os: [ubuntu-24.04, macos-14, windows-2025]
node-version: ['22']
node-version: ['24']
install-command: ['npm ci']
machine: ['0', '1', '2', '3', '4']
include:
Expand Down Expand Up @@ -168,7 +168,7 @@ jobs:
# Use npm@10 on Node 22+ due to https://github.com/npm/cli/issues/8489
- name: Setup npm version
run: npm install -g npm@10
if: ${{ !steps.release-check.outputs.IS_RELEASE && matrix.node-version == '22' }}
if: ${{ !steps.release-check.outputs.IS_RELEASE && matrix.node-version != '18.14.0' }}
- name: corepack update
# corepack version distributed with Node.js has a problem with new package manager releases,
# so forcing fixed version of corepack here
Expand Down
Loading