Skip to content

Conversation

@jakebailey
Copy link
Member

If a job fails in a merge queue, it's either a platform flake, or a runtime merge conflict. Don't bother running all jobs and let it quit out as early as possible.

Copilot AI review requested due to automatic review settings December 8, 2025 18:59
Copy link
Contributor

Copilot AI left a comment

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 conditionally enables fail-fast behavior in CI workflows specifically for merge queue runs. The rationale is that failures in merge queues are typically either platform flakes or runtime merge conflicts, so there's no value in continuing to run all jobs when one fails.

Key Changes

  • Modified fail-fast setting in the test job to be true only when github.event_name == 'merge_group'
  • Applied the same conditional logic to the lint job for consistency

@jakebailey jakebailey enabled auto-merge December 8, 2025 19:06
@jakebailey jakebailey added this pull request to the merge queue Dec 8, 2025
Merged via the queue into main with commit 7c0aa9e Dec 8, 2025
28 checks passed
@jakebailey jakebailey deleted the jabaile/merge-group-fail-fast branch December 8, 2025 19:34
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.

3 participants