Skip to content

Conversation

@electron-roller
Copy link
Contributor

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.

@electron-roller electron-roller bot requested a review from a team as a code owner June 21, 2024 13:00
@electron-roller electron-roller bot added the semver/minor backwards-compatible functionality label Jun 21, 2024
@codebytere codebytere self-assigned this Jun 22, 2024
@codebytere codebytere requested a review from a team as a code owner June 24, 2024 07:50
@codebytere codebytere added no-backport semver/patch backwards-compatible bug fixes labels Jun 24, 2024
@codebytere codebytere added api-review/requested 🗳 and removed semver/minor backwards-compatible functionality labels Jun 24, 2024
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Jun 24, 2024
@codebytere codebytere added new-pr 🌱 PR opened recently and removed api-review/requested 🗳 labels Jun 24, 2024
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Jun 24, 2024
// 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;
Copy link
Member

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?

Copy link
Member

@codebytere codebytere Jun 25, 2024

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

@codebytere codebytere merged commit ec24b51 into main Jun 27, 2024
@release-clerk
Copy link

release-clerk bot commented Jun 27, 2024

Release Notes Persisted

Updated Node.js to v20.15.0.

@codebytere codebytere deleted the roller/node/main branch June 27, 2024 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-backport semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants