Skip to content

Commit 390508c

Browse files
committed
ci: add final ci gate job (following patchloom/attune pattern)
This provides a single always() job that the required checks / branch protection can depend on, while the matrix jobs use the changes filter. This is exactly how patchloom structures its final "ci" job. Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
1 parent d4746ea commit 390508c

1 file changed

Lines changed: 16 additions & 0 deletions

File tree

.github/workflows/ci.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -180,3 +180,19 @@ jobs:
180180
- name: Run UI tests
181181
if: runner.os != 'Linux'
182182
run: npx extest run-tests './out-uitest/test/ui/*.test.js' --extensions_dir .vscode-test/extensions
183+
184+
ci:
185+
# Final gate job (if: always()). This is the recommended pattern (see patchloom
186+
# and attune) so that individual matrix jobs can use the changes filter without
187+
# breaking required status checks or Scorecard parsing.
188+
if: always()
189+
needs:
190+
- unit-test
191+
- build
192+
- integration-test
193+
runs-on: ubuntu-latest
194+
timeout-minutes: 5
195+
steps:
196+
- name: All CI jobs passed (or were correctly skipped for docs-only change)
197+
run: |
198+
echo "CI gate passed."

0 commit comments

Comments
 (0)