Skip to content

Conversation

@sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Dec 13, 2024

Also convert to check_expected_size_in_file which allows this test
to be updated via --rebaseline

This change shows the incremental progress we've been making on code size recently!

@sbc100 sbc100 requested review from dschuff and kripken December 13, 2024 19:23
@sbc100 sbc100 force-pushed the rebaseline_browser_test branch from 36af7db to 19c263a Compare December 13, 2024 19:24
Also convert to `check_expected_size_in_file` which allows this test
to be updated via `--rebaseline`

This change shows the incremental progress we've been making on code
size recently!
@sbc100 sbc100 force-pushed the rebaseline_browser_test branch from 19c263a to ab5297b Compare December 13, 2024 19:30
@sbc100
Copy link
Collaborator Author

sbc100 commented Dec 13, 2024

I might land this TBR since its currently causing CI to fail.

@sbc100 sbc100 merged commit a8d325c into emscripten-core:main Dec 13, 2024
12 of 28 checks passed
@sbc100 sbc100 deleted the rebaseline_browser_test branch December 13, 2024 20:00
@dschuff
Copy link
Member

dschuff commented Dec 13, 2024

LGTM, thanks

hedwigz pushed a commit to hedwigz/emscripten that referenced this pull request Dec 18, 2024
Also convert to `check_expected_size_in_file` which allows this test
to be updated via `--rebaseline` 

This change shows the incremental progress we've been making on code
size recently!
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