Skip to content

ci: add required-ci aggregator job#377

Merged
MarshallOfSound merged 2 commits intomainfrom
sam/required-ci-job
Mar 25, 2026
Merged

ci: add required-ci aggregator job#377
MarshallOfSound merged 2 commits intomainfrom
sam/required-ci-job

Conversation

@MarshallOfSound
Copy link
Copy Markdown
Member

Summary

  • Add a Required CI job that needs: test and runs with if: always()
  • Fails the run if the test job result is anything other than success or skipped
  • Gives branch protection a single, stable status check to require instead of listing every matrix variant

Test plan

  • CI passes on this PR
  • Required CI reports success when all test matrix jobs pass
  • Required CI reports failure when any test matrix job fails

Adds a single "Required CI" job that depends on the test matrix and
fails if any test variant did not succeed. This lets branch protection
target one stable check name instead of every matrix combination.
@MarshallOfSound MarshallOfSound requested a review from a team as a code owner March 25, 2026 23:30
@MarshallOfSound MarshallOfSound merged commit f759a68 into main Mar 25, 2026
7 checks passed
@MarshallOfSound MarshallOfSound deleted the sam/required-ci-job branch March 25, 2026 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants