-
Notifications
You must be signed in to change notification settings - Fork 16.9k
chore: upgrade to Node.js v14.9.0 #25249
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
Conversation
17156aa to
5b5c169
Compare
182dd3f to
799c17e
Compare
|
This PR is ready for initial review - arm CI is experiencing some Struggles™️ so i'll sort those out before looking to land this. |
79a00ae to
a3a43b9
Compare
This was previously broken - to make it work we need to figure out how to initialize inspector before RunBootstrapping() and we can do that as a follow-up
This behavior is no longer valid as of nodejs/node#29197
a3a43b9 to
6e94a57
Compare
6e94a57 to
2f7058c
Compare
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.
Blocking on the types problem as that will be a big regression for folks if some types stop being auto-inferred 🤔
Happy to help look into it once I'm not fighting nan in the upgrades branch
|
@MarshallOfSound it's been resolved - just working on verifying one last thing with the net specs atm |
015c1de to
73ba23b
Compare
73ba23b to
28bc637
Compare
nornagon
left a comment
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 with nits
|
Release Notes Persisted
|
|
Is this added to the v11 beta? This fixes an important issue about worker_thread and we hope we can adapt this. |
Description of Change
Upgrade Node.js to
v14.9.0Checklist
npm testpassesRelease Notes
Notes: Upgraded Node.js to v14.9.0.