From 96efff6572b15a0c2b6b2ff9bce5c639c8dfb9ab Mon Sep 17 00:00:00 2001 From: Nate Moore Date: Sat, 6 Jun 2026 12:13:58 -0500 Subject: [PATCH] fix(ci): rebuild from clean in size-report so build-config changes are 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. --- .github/workflows/size-report.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/size-report.yml b/.github/workflows/size-report.yml index 1d8ce71..3e02218 100644 --- a/.github/workflows/size-report.yml +++ b/.github/workflows/size-report.yml @@ -35,7 +35,7 @@ jobs: node-version: 24 - name: build (head) - run: make && deno task build:npm 0.0.0 + run: make clean && make && deno task build:npm 0.0.0 - name: measure sizes (head) run: | @@ -54,7 +54,7 @@ jobs: git submodule update --init --recursive - name: build (base) - run: make && deno task build:npm 0.0.0 + run: make clean && make && deno task build:npm 0.0.0 - name: measure sizes (base) run: deno run --allow-read /tmp/measure-size.ts > /tmp/base-sizes.json