-
-
Notifications
You must be signed in to change notification settings - Fork 17
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: Use Node 22 #253
chore: Use Node 22 #253
Conversation
@@ -179,7 +179,7 @@ async function expectPullRequestTemplate(workspace, workspaceName) { | |||
|
|||
const pullRequestTemplate = await getWorkspaceFile( | |||
workspace, | |||
'.github/PULL_REQUEST_TEMPLATE.md', | |||
'.github/pull_request_template.md', |
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.
Notably, whatever fs
call is backing this is newly case-sensitive in Node 22, so this fails if it doesn't actually match the filename.
Why not wait until it's actually https://github.com/nodejs/release#release-schedule Should be in ~1 month. |
Sure, we can wait for a month. |
Node.js 22.x is LTS as of today. |
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.
LGTM!
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.
Looks good!
As of October 29, Node.js 22 is the active
lts
version, and we should start to configure our repositories to use it as the latest supported version. This also updates our minimum supported Node versions for 18 and 20.