Skip to content

Add MIN_NODE_VERSION setting#18521

Merged
sbc100 merged 1 commit intomainfrom
min_node_version
Jan 20, 2023
Merged

Add MIN_NODE_VERSION setting#18521
sbc100 merged 1 commit intomainfrom
min_node_version

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented Jan 14, 2023

This, along with #18465 (which run tests on the oldest supported version of node) should pave the way for us to update the emsdk version node to something a little more modern.

See emscripten-core/emsdk#829

This first thing I do with this setting is use it do disable NODEJS_CATCH_REJECTION by default when we are targeting node 15 and above.

@sbc100 sbc100 force-pushed the min_node_version branch 3 times, most recently from 0302004 to f793dc7 Compare January 19, 2023 18:24
@sbc100 sbc100 requested review from dschuff and kripken January 19, 2023 18:24
@sbc100 sbc100 enabled auto-merge (squash) January 20, 2023 20:27
Copy link
Copy Markdown
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm % comments

This, along with #18465 (which run tests on the oldest supported
version of node) should pave the way for us to update the emsdk version
node to something a little more modern.

See emscripten-core/emsdk#829

This first thing I do with this setting is use it do disable
`NODEJS_CATCH_REJECTION` by default when we are targeting node 15 and
above.
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