Skip to content

ci: add extra final step to aggregate matrix results#44

Open
shadaj wants to merge 1 commit into
mainfrom
ci-final-step
Open

ci: add extra final step to aggregate matrix results#44
shadaj wants to merge 1 commit into
mainfrom
ci-final-step

Conversation

@shadaj

@shadaj shadaj commented Jun 26, 2026

Copy link
Copy Markdown
Member

Otherwise, when intermediate steps are skipped due to no changes, the PR
hangs in a state where it is waiting for expected checks.

Otherwise, when intermediate steps are skipped due to no changes, the PR
hangs in a state where it is waiting for expected checks.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 26, 2026

Copy link
Copy Markdown

Deploying infinity with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9d6a2b8
Status: ✅  Deploy successful!
Preview URL: https://66ef5bf3.infinity-dc7.pages.dev
Branch Preview URL: https://ci-final-step.infinity-dc7.pages.dev

View logs

@shadaj shadaj marked this pull request as ready for review June 26, 2026 23:32
@MingweiSamuel MingweiSamuel requested a review from Copilot July 6, 2026 22:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds an always-running “aggregator” job to produce a single, deterministic check result for the full test matrix, even when the matrix job is skipped (e.g., due to the skip-duplicate-actions gate). This helps avoid PRs getting stuck “waiting for expected checks” when matrix jobs don’t execute.

Changes:

  • Add a new test-all job that always runs after test and converts needs.test.result into a pass/fail outcome.
  • Treat success and skipped as passing outcomes for the aggregate check.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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