-
Notifications
You must be signed in to change notification settings - Fork 16.9k
chore: bump node to v20.15.0 (main) #42616
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
34f5f48 to
5e97f8c
Compare
5e97f8c to
a43d7ea
Compare
| // V8::EnableWebAssemblyTrapHandler can be called only once or it will | ||
| // hard crash. We need to prevent Node.js calling it in the event it has | ||
| // already been called. | ||
| node::per_process::cli_options->disable_wasm_trap_handler = true; |
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.
Should this be true or some kind of "is renderer" check as I assume the main process + utility process's don't have this set up?
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 yeah we always want it - if you look at the diff you'll see it was true before but we patched it in as an env option before joyee merged a PR to accomplish the same goal
18608f8 to
b135e29
Compare
b135e29 to
a2a64e9
Compare
|
Release Notes Persisted
|
Updating Node.js to v20.15.0.
See all changes in v20.14.0..v20.15.0
Notes: Updated Node.js to v20.15.0.