Skip to content

Use run_codesize_test helper for test_small_js_flags. NFC#27030

Merged
sbc100 merged 1 commit into
emscripten-core:mainfrom
sbc100:test_small_js_flags
May 29, 2026
Merged

Use run_codesize_test helper for test_small_js_flags. NFC#27030
sbc100 merged 1 commit into
emscripten-core:mainfrom
sbc100:test_small_js_flags

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented May 29, 2026

This allows is to track the full JS output for this test.

I also ended up changing the input source to our standard hello world one. I don't think this test was concerned with the details of the particular input source file here.

@sbc100 sbc100 requested a review from kripken May 29, 2026 18:04
Comment thread test/test_codesize.py
Comment thread test/codesize/test_small_js_flags.json Outdated
@sbc100 sbc100 force-pushed the test_small_js_flags branch 2 times, most recently from 51cfda8 to e83340e Compare May 29, 2026 18:54
Comment thread test/test_codesize.py Outdated
This allows is to track the full JS output for this test.
@sbc100 sbc100 force-pushed the test_small_js_flags branch from e83340e to 6007181 Compare May 29, 2026 20:23
@sbc100 sbc100 requested a review from kripken May 29, 2026 20:24
@sbc100 sbc100 merged commit a51e7ef into emscripten-core:main May 29, 2026
18 of 21 checks passed
@sbc100 sbc100 deleted the test_small_js_flags branch May 29, 2026 20:40
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