Skip to content

Comments

Remove NODEJS_CATCH_REJECTION#26330

Merged
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:NODEJS_CATCH_REJECTION
Feb 24, 2026
Merged

Remove NODEJS_CATCH_REJECTION#26330
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:NODEJS_CATCH_REJECTION

Conversation

@sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Feb 24, 2026

This settings only has an effect when targeting very old version of node (< 15). Its trivial to replace with a simple --pre-js file or with the --unhandled-rejections=strict command line flag which is basically emulates.

Version of node above v15 have this behavior by default.

@sbc100 sbc100 requested review from dschuff and kripken February 24, 2026 01:56
@sbc100 sbc100 force-pushed the NODEJS_CATCH_REJECTION branch from 87fe918 to abb3975 Compare February 24, 2026 01:58
This settings only has an effect when targeting very old version of node
(< 15).  Its trivial to replace with a simple `--pre-js` file or with
the `--unhandled-rejections=strict` command line flag which is basically
emulates.

Version of node above v15 have this behavior by default.
@sbc100 sbc100 force-pushed the NODEJS_CATCH_REJECTION branch from abb3975 to ba4adcb Compare February 24, 2026 18:35
@sbc100 sbc100 enabled auto-merge (squash) February 24, 2026 18:42
@sbc100 sbc100 merged commit 8407826 into emscripten-core:main Feb 24, 2026
36 checks passed
@sbc100 sbc100 deleted the NODEJS_CATCH_REJECTION branch February 24, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants