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.
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
ci: test on Node.js 22 #8010
base: main
Are you sure you want to change the base?
ci: test on Node.js 22 #8010
Conversation
@@ -17,7 +17,7 @@ jobs: | |||
node: | |||
- '18.12' | |||
- '20' | |||
- '21' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't want to test against v21?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v21 is not an lts version. When 22 is out, v21 is basically deprecated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as you can see, we don't test v19 as well.
@@ -43,8 +43,8 @@ jobs: | |||
with: | |||
node-version: ${{ matrix.node }} | |||
cache: 'pnpm' | |||
- name: Install npm@7 | |||
run: npm add --global npm@7 | |||
- name: Install npm@10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
npm v10 appears to require a minimum version of Node v20.
Do we want to do a matrix of node-version
and npc-version
to accommodate for this?
No description provided.