Add a build (all) gate job as the single required check - #64
Closed
ppkarwasz wants to merge 1 commit into
Closed
Conversation
Aggregate every build job into one slim gate that fails when a gating build failed, so branch protection can require a single stable check and the matrix can change shape without touching the required-checks list. A leg with continue-on-error reports success to needs and stays advisory; without always() a failed leg would leave the gate skipped, which branch protection treats as satisfied, so the gate runs always and fails explicitly. Assisted-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MbfpcsKig16HfSe6jGhcvh
ppkarwasz
marked this pull request as draft
August 31, 2026 11:41
Member
Author
|
Converting to draft, since this only makes sense if we require PRs too. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a
build (all)aggregate job onubuntu-slimthatneeds:every build job and fails when any gating build failed, so branch protection can require this one stable check instead of one per matrix leg (14 today, renamed whenever the matrix changes).Two GitHub Actions semantics drive the shape of the job:
continue-on-error: truereports success to theneedscontext, so such legs are invisible to the gate and stay purely advisory (currently the GraalVM native leg and, until the Android suite fix lands, the Android legs).if: always(), a failed leg would leave the gate skipped, and branch protection treats a skipped required check as satisfied; the gate therefore always runs and fails explicitly (cancelledincluded, so a half-cancelled run cannot green-light a merge).After merging, mark
build (all)as the required status check in the branch protection settings. Ifubuntu-slimis not available for this repository, the fallback is a one-word change toubuntu-latest.🤖 Generated with Claude Code
https://claude.ai/code/session_01MbfpcsKig16HfSe6jGhcvh