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
Update GHA to node 20 #1022
Update GHA to node 20 #1022
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@@ -1 +1 @@ | |||
v14 | |||
v16 |
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 are running CI tests on Node 16, so we should also set nvm to at least 16 as well.
- name: Setup Node.js 16.x | ||
uses: actions/setup-node@64ed1c7eab4cce3362f8c340dee64e5eaeef8f7c # v3.6.0 | ||
uses: actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8 # v4.0.2 | ||
with: | ||
node-version: 16.x |
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.
Would these instances of 16.x
need to be updated to 20.x
as well?
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.
Thankfully we do not, we can still use Node 20 > to build things during a GHA. Just no GHA that uses Node 20 > as it's runner.
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.
Ah, important distinction 👍 Thanks!
🎟️ Update GHA to Node 20
Description
Updates all offending GHA steps to Node 20.