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: switch workflows to use node 20 and 22 by default #5046

Merged
merged 6 commits into from
Oct 11, 2024
Merged

Conversation

marikaner
Copy link
Contributor

@marikaner marikaner commented Sep 30, 2024

Node 22 is the current node version, so all non-matrix workflows should run on this version by default.
Github actions seems to only allow node16 and node20 according to issues and docs that I found. I didn't try 22 though.

The tests are now running on all three versions.

@marikaner marikaner changed the title support node 20 chore: switch workflows to use node 20 and 22 by default Oct 11, 2024
@marikaner marikaner marked this pull request as ready for review October 11, 2024 10:57
@@ -14,7 +14,7 @@ jobs:
- run: git fetch --depth=1
- uses: actions/setup-node@v4
with:
node-version: 18
node-version: 22
cache: 'yarn'
- run: yarn install --frozen-lockfile
Copy link
Contributor

Choose a reason for hiding this comment

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

--ignore-engines here as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes! good catch

@marikaner marikaner enabled auto-merge (squash) October 11, 2024 12:19
@marikaner marikaner merged commit 97d1b69 into main Oct 11, 2024
15 checks passed
@marikaner marikaner deleted the node20 branch October 11, 2024 12:27
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