-
Notifications
You must be signed in to change notification settings - Fork 6
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
[NT] Support NodeJS 20 #572
Conversation
deeefb8
to
e2224cd
Compare
// Sleep briefly to allow the OS to free up the port so that we can re-bind it again in the | ||
// next test. | ||
await new Promise((r) => setTimeout(r, 1000)); |
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.
This was needed to get the tests to pass on the NodeJS 20 image. Unsure what specifically changed there to need this change, but this fixed all of the issues.
@@ -0,0 +1 @@ | |||
v20.10.0 |
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.
There wasn't previously an .nvmrc
file here. Added one.
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 👍
Add support for NodeJS 20.