Skip to content
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

chore: test on Node.js 20 #22651

Merged
merged 2 commits into from
May 30, 2023
Merged

Conversation

mxschmitt
Copy link
Member

@mxschmitt mxschmitt commented Apr 26, 2023

As per offline discussion we keep the majority of our bots on Node.js 16 for now, since thats where most of our users are.

This patch does remove Node.js 14 bots, since 14 goes EOL before our next release, see here and adds Node.js 20 bots.

Node.js 20 tests are failing since nodejs/node@5ec0f39 so we had to adjust TestServer a bit.

Fixes #22582

@mxschmitt mxschmitt force-pushed the devops/test-node-20 branch 2 times, most recently from d63a5f3 to 133a837 Compare April 26, 2023 13:50
@benmccann
Copy link
Contributor

Just FYI, the SvelteKit CI is failing on Node 20.0.0, but one of the maintainers figured out it works with the latest nightly release, so perhaps the Node 20 tests here will start passing after the next release of Node 20.

@mxschmitt mxschmitt force-pushed the devops/test-node-20 branch from 133a837 to 8cc3e70 Compare May 16, 2023 19:11
@mxschmitt mxschmitt added the CQ1 label May 16, 2023
@mxschmitt mxschmitt force-pushed the devops/test-node-20 branch from 8cc3e70 to e05afea Compare May 25, 2023 17:42
@mxschmitt mxschmitt added CQ1 and removed CQ1 labels May 25, 2023
@mxschmitt mxschmitt force-pushed the devops/test-node-20 branch from e05afea to bf889d7 Compare May 27, 2023 09:00
@mxschmitt mxschmitt changed the title devops: test on Node.js 20 chore: test on Node.js 20 May 27, 2023
- os: ubuntu-latest
node_version: "^16.0.0"
Copy link
Member Author

Choose a reason for hiding this comment

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

note: 16 is already tested in tests_primary

@mxschmitt mxschmitt added CQ1 and removed CQ1 labels May 27, 2023
@mxschmitt mxschmitt force-pushed the devops/test-node-20 branch from bf889d7 to 335677a Compare May 27, 2023 09:24
@mxschmitt mxschmitt added CQ1 and removed CQ1 labels May 27, 2023
@mxschmitt mxschmitt merged commit 1f7223e into microsoft:main May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] add Node 20 to CI matrix
3 participants