Skip to content

Conversation

@sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Oct 9, 2025

Prior to this we were tacking on the extra misc tests on the end of wasm64-4gb runner.

@sbc100 sbc100 requested review from dschuff and kripken October 9, 2025 17:20
Prior to this we were tacking on the extra misc tests on the end of
wasm64-4gb runner.
@kripken
Copy link
Member

kripken commented Oct 9, 2025

What is the advantage to this split? Is it faster overall?

@sbc100
Copy link
Collaborator Author

sbc100 commented Oct 9, 2025

What is the advantage to this split? Is it faster overall?

Its likely fast yes, but mostly its just organizational. I don't like lumping in these misc tests with the runner called _4gb (which mostly does run just the 4gb test suite).

Copy link
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.

Ok, lgtm, but maybe measure to make sure this isn't wasteful (maybe there are so few misc tests that the general job overhead is high compared to them).

@sbc100
Copy link
Collaborator Author

sbc100 commented Oct 9, 2025

Ok, lgtm, but maybe measure to make sure this isn't wasteful (maybe there are so few misc tests that the general job overhead is high compared to them).

This job takes 5 mins to run, and only a few seconds to setup so I think its worth it.

@sbc100 sbc100 merged commit 6ec477e into emscripten-core:main Oct 9, 2025
34 checks passed
@sbc100 sbc100 deleted the node_misc branch October 9, 2025 20:11
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