From 168185ae2b791944337c5a1a4b6a3921223d1591 Mon Sep 17 00:00:00 2001 From: Rajat Ahuja Date: Wed, 26 Aug 2026 15:04:03 -0400 Subject: [PATCH] fix(ci): stop skipping unified tests on merge to main (#1083) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The gate job only runs on `pull_request: labeled`, so it is skipped on push. A skipped ancestor propagates down the needs chain unless a job opts out, which `unified-tests` never did — so the suite has been skipped on every merge to main while still burning a Fly machine. --- .github/workflows/unified-tests.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/unified-tests.yml b/.github/workflows/unified-tests.yml index 4813a126..b73bda32 100644 --- a/.github/workflows/unified-tests.yml +++ b/.github/workflows/unified-tests.yml @@ -57,7 +57,10 @@ jobs: name: Run Unified Tests runs-on: ${{ fromJSON(format('[{0}]', needs.start-runner.outputs.runner-labels)) }} needs: start-runner - if: needs.start-runner.outputs.runner-ready == 'true' + # !cancelled() so this doesn't inherit gate's skip on push events. + if: >- + !cancelled() && + needs.start-runner.outputs.runner-ready == 'true' timeout-minutes: 90 environment: unified-tests permissions: