From aed850d3e645a36710d66ebc2bb10af1d7aab73e Mon Sep 17 00:00:00 2001 From: Arran Schlosberg Date: Thu, 10 Sep 2026 15:45:21 +0100 Subject: [PATCH 1/2] ci: re-package and confirm `dist` is up to date with `src` --- .github/workflows/test.yml | 27 ++++++++++++++++++++++++++- src/index.ts | 2 ++ 2 files changed, 28 insertions(+), 1 deletion(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 696c92d..f664455 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -8,14 +8,38 @@ on: workflow_dispatch: jobs: + check_dist: + name: dist-up-to-date + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v5 + + - uses: actions/setup-node@v6 + with: + node-version: '20' # matches `runs.using` in action.yml + cache: npm + + - run: npm ci + - run: npm run package + + - name: Check `dist` matches `src` + # `dist` is committed because that's what `action.yml` runs, so it + # MUST be rebuilt whenever `src` changes. + run: | + if ! git --no-pager diff --exit-code -- dist; then + echo '::error::dist is stale; run `npm run package` and commit the result.' >&2 + exit 1 + fi + test: runs-on: ubuntu-latest name: test-all # PRs can require just this job to pass, regardless of matrix changes below - needs: [test_matrix, test_multi] + needs: [check_dist, test_matrix, test_multi] steps: - run: echo "Test-matrix successful" # anything with zero exit code test_matrix: + needs: check_dist strategy: matrix: os: [macos-15-intel, ubuntu-latest] # arm64 and Windows aren't supported @@ -40,6 +64,7 @@ jobs: run: solc --version | grep "${{matrix.solc}}" test_multi: + needs: check_dist name: setup-solc-multi-version runs-on: ubuntu-latest steps: diff --git a/src/index.ts b/src/index.ts index 6f84f44..023c26e 100644 --- a/src/index.ts +++ b/src/index.ts @@ -53,6 +53,8 @@ try { console.info(`${version} at ${outs}`); }; + console.info("DO NOT MERGE THIS LINE"); + } catch (error: any) { if (error instanceof Error) { core.setFailed(error.message) From 38762fa6d9c9b940607cfd260937a6cfc522e97b Mon Sep 17 00:00:00 2001 From: Arran Schlosberg Date: Thu, 10 Sep 2026 15:48:37 +0100 Subject: [PATCH 2/2] chore: remove deliberate break that exercised new CI job --- src/index.ts | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/index.ts b/src/index.ts index 023c26e..6f84f44 100644 --- a/src/index.ts +++ b/src/index.ts @@ -53,8 +53,6 @@ try { console.info(`${version} at ${outs}`); }; - console.info("DO NOT MERGE THIS LINE"); - } catch (error: any) { if (error instanceof Error) { core.setFailed(error.message)