Skip to content

Show diff in codesize expectations CI job - #27763

Merged
sbc100 merged 1 commit into
emscripten-core:mainfrom
hoodmane:codesize-job-show-diff
Sep 22, 2026
Merged

sbc100 merged 1 commit into
emscripten-core:mainfrom
hoodmane:codesize-job-show-diff

Conversation

@hoodmane

Copy link
Copy Markdown
Collaborator

I tried it here and it shows the following exactly as I wanted:

Details

Full diff from the current expectations to the rebaselined expectations:

diff --git a/test/codesize/test_codesize_files_js_fs.json b/test/codesize/test_codesize_files_js_fs.json
index 4e36e55..919b88c 100644
--- a/test/codesize/test_codesize_files_js_fs.json
+++ b/test/codesize/test_codesize_files_js_fs.json
@@ -1,10 +1,10 @@
 {
-  "a.out.js": 17871,
-  "a.out.js.gz": 7460,
+  "a.out.js": 17895,
+  "a.out.js.gz": 7467,
   "a.out.nodebug.wasm": 381,
   "a.out.nodebug.wasm.gz": 258,
   "total": 18276,
-  "total_gz": 7718,
+  "total_gz": 7725,
   "sent": [
     "a (fd_write)",
     "b (fd_read)",
diff --git a/test/codesize/test_codesize_hello_dylink_all.json b/test/codesize/test_codesize_hello_dylink_all.json
index 00cc3bd..0b513f9 100644
--- a/test/codesize/test_codesize_hello_dylink_all.json
+++ b/test/codesize/test_codesize_hello_dylink_all.json
@@ -1,5 +1,5 @@
 {
-  "a.out.js": 270695,
+  "a.out.js": 270707,
   "a.out.nodebug.wasm": 588098,
   "total": 858805,
   "sent": [

@sbc100
sbc100 merged commit 8c64a60 into emscripten-core:main Sep 22, 2026
22 of 24 checks passed
@hoodmane
hoodmane deleted the codesize-job-show-diff branch September 22, 2026 19:29
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