Skip to content

fix(ci): rebuild from clean in size-report so build-config changes are measured#90

Merged
natemoo-re merged 1 commit into
mainfrom
fix/size-report-rebuild
Jun 17, 2026
Merged

fix(ci): rebuild from clean in size-report so build-config changes are measured#90
natemoo-re merged 1 commit into
mainfrom
fix/size-report-rebuild

Conversation

@natemoo-re

Copy link
Copy Markdown
Member

The size report builds head then base without cleaning in between. For PRs that change only build configuration (Makefile, patches/, compiler flags) and not src/*.c, make sees clayterm.wasm as newer than the unchanged sources and skips the base rebuild — so the base step re-measures the head artifact, the delta comes out 0, and no size comment is posted even though the wasm size actually changed.

…e measured

The size report builds head then base without cleaning between them. For PRs that change only build config (Makefile, patches/, flags) and not src/*.c, make sees clayterm.wasm as newer than the unchanged sources and skips the base rebuild, so base re-measures head's artifact, delta is 0, and no size comment is posted. Run make clean && make for both builds so each is from scratch.
@pkg-pr-new

pkg-pr-new Bot commented Jun 6, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/clayterm@90

commit: 96efff6

@codspeed-hq

codspeed-hq Bot commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Merging this PR will degrade performance by 40.53%

❌ 2 (👁 2) regressed benchmarks
✅ 21 untouched benchmarks

Performance Changes

Mode Benchmark BASE HEAD Efficiency
👁 Simulation long input burst (200 bytes) 576.1 µs 1,265.1 µs -54.46%
👁 Simulation diff render (second frame) 690.4 µs 888.9 µs -22.33%

Comparing fix/size-report-rebuild (96efff6) with main (48ff4bb)

Open in CodSpeed

@natemoo-re natemoo-re merged commit 72890de into main Jun 17, 2026
12 checks passed
@natemoo-re natemoo-re deleted the fix/size-report-rebuild branch June 17, 2026 02:06
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