From 3e01e738c7c1e32295d6b34d39562a583e50d1fa Mon Sep 17 00:00:00 2001 From: Derek Lewis Date: Tue, 18 Aug 2026 05:58:21 +0000 Subject: [PATCH] =?UTF-8?q?=F0=9F=8F=97=EF=B8=8F=F0=9F=94=A7=EF=BC=9Alint?= =?UTF-8?q?=20live,=20and=20let=20the=20badge=20say=20so?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The badge reported whichever branch ran last. A pull-request-only workflow leaves the default branch with no status, so a contributor's failing pull request turned the front page red while live was fine. Nothing checked live either. The queue squashes without rebuilding, so two changes that passed separately could land together and break something, with only the deploy having run. The path filters stay for pull requests. On live the whole suite runs. Signed-off-by: Derek Lewis Assisted-by: Claude-Code:claude-opus-5 --- .github/workflows/lint-and-test.yml | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index 4bb6bcffe..6c01dd6db 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -1,7 +1,14 @@ name: Lint and test -# By default, runs when a pull request is opened, synchronized, or reopened. -on: pull_request +# Also on live, for two reasons: a pull-request-only workflow leaves the +# default branch with no status, so the README badge reports whichever branch +# ran last; and the queue squashes without rebuilding, so nothing else sees +# what actually landed. +on: + pull_request: + push: + branches: + - live jobs: lint_and_test: @@ -119,7 +126,8 @@ jobs: if: ${{ steps.filter.outputs.yaml == 'true' }} run: nps verify.yaml - # Only run tests if the PR touches behavior-related files. + # Only run tests if the PR touches behavior-related files. On live + # everything runs, since nothing else checks what landed. - name: Test - if: ${{ steps.filter.outputs.js == 'true' || steps.filter.outputs.json == 'true' || steps.filter.outputs.ts == 'true' }} + if: ${{ github.event_name == 'push' || steps.filter.outputs.js == 'true' || steps.filter.outputs.json == 'true' || steps.filter.outputs.ts == 'true' }} run: nps test