fix(ci): add detect to all-checks-pass needs so its failure blocks merge

If detect fails, all downstream sub-workflows get SKIPPED (they have
needs: detect). all-checks-pass used if: always() and only checked the
sub-workflows — which all showed as 'skipped' (= success) — so it passed
even though the root cause (detect) failed. This made the PR mergeable
despite a broken CI pipeline.

Add detect to all-checks-pass needs so its failure propagates to the
gate job and blocks the merge.
This commit is contained in:
ethernet 2026-07-20 17:08:02 -04:00
parent 1f76bdc5b2
commit 5c7993ec60

View file

@ -259,6 +259,7 @@ jobs:
all-checks-pass:
name: All required checks pass
needs:
- detect
- tests
- lint
- js-tests