ci: Add consolidated check job so we can change branch protection rules (#22732)

This commit is contained in:
Declan Carroll
2025-12-04 12:01:52 +00:00
committed by GitHub
parent 599c6ebe98
commit 31e5b4590b
2 changed files with 14 additions and 6 deletions

View File

@@ -5,3 +5,4 @@ self-hosted-runner:
- blacksmith-2vcpu-ubuntu-2204-arm
- blacksmith-4vcpu-ubuntu-2204-arm
- blacksmith-8vcpu-ubuntu-2204
- ubuntu-slim

View File

@@ -135,12 +135,19 @@ jobs:
uses: ./.github/workflows/playwright-test-reusable.yml
secrets: inherit
e2e-checks:
name: E2E - Checks
runs-on: ubuntu-latest
needs: [e2e-test]
# This job is required by GitHub branch protection rules.
# PRs cannot be merged unless this job passes.
# If you add/remove jobs that should block merging, update the 'needs' array below.
required-checks:
name: Required Checks
needs: [install-and-build, unit-test, typecheck, lint, e2e-test]
if: always()
runs-on: ubuntu-slim
steps:
- name: Fail if E2E tests failed
if: needs.e2e-test.result == 'failure'
- name: Fail if any required job failed or was skipped unexpectedly
# The non_python_changed check allows jobs to be skipped for python-only changes,
# since those jobs don't run when only python files are modified.
if: |
contains(needs.*.result, 'failure') ||
(needs.install-and-build.outputs.non_python_changed == 'true' && contains(needs.*.result, 'skipped'))
run: exit 1