Skip to content

ci: Add aggregator job for test matrix#575

Merged
RandomByte merged 1 commit into
mainfrom
ci/aggregate-test-matrix
Jul 1, 2026
Merged

ci: Add aggregator job for test matrix#575
RandomByte merged 1 commit into
mainfrom
ci/aggregate-test-matrix

Conversation

@RandomByte

Copy link
Copy Markdown
Member

Add a 'Tests' job that depends on both matrix jobs (test and
test-ui5cli-matrix) so branch protection can require a single check
instead of every matrix combination. The matrix rows remain visible in
the PR checks list; only the required-check surface collapses.

'if: always()' ensures the aggregator still runs when matrix cells fail,
so a failure surfaces as 'failed' rather than 'skipped'.

Add a 'Tests' job that depends on both matrix jobs (test and
test-ui5cli-matrix) so branch protection can require a single check
instead of every matrix combination. The matrix rows remain visible in
the PR checks list; only the required-check surface collapses.

'if: always()' ensures the aggregator still runs when matrix cells fail,
so a failure surfaces as 'failed' rather than 'skipped'.
@RandomByte RandomByte requested a review from a team July 1, 2026 11:46
@RandomByte RandomByte merged commit 82e9b34 into main Jul 1, 2026
18 checks passed
@RandomByte RandomByte deleted the ci/aggregate-test-matrix branch July 1, 2026 13:54
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