Skip to content

Move "Patches Apply Cleanly" check to the top#2197

Merged
dagood merged 1 commit intomicrosoft/mainfrom
dev/mclayton/fix-pipeline-order
Mar 19, 2026
Merged

Move "Patches Apply Cleanly" check to the top#2197
dagood merged 1 commit intomicrosoft/mainfrom
dev/mclayton/fix-pipeline-order

Conversation

@michelle-clayton-work
Copy link
Contributor

In response to #2148

@michelle-clayton-work michelle-clayton-work force-pushed the dev/mclayton/fix-pipeline-order branch from 2765303 to 6a995d0 Compare March 19, 2026 03:37
@michelle-clayton-work michelle-clayton-work marked this pull request as ready for review March 19, 2026 16:26
@michelle-clayton-work michelle-clayton-work requested a review from a team as a code owner March 19, 2026 16:26
Copilot AI review requested due to automatic review settings March 19, 2026 16:26
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

Updates the GitHub Actions workflow name so the “Patches Apply Cleanly”/test-related check appears at the top of the PR checks list, improving visibility of early failures (per #2148).

Changes:

  • Renamed the workflow from "Test" to "0: Test" to force sort ordering in the PR checks UI.
  • Added an inline comment documenting why the "0:" prefix is used.

You can also share your feedback on Copilot code review. Take the survey.

Copy link
Member

@dagood dagood left a comment

Choose a reason for hiding this comment

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

LGTM, good comment. Opened #2199 to exercise it and take a few screenshots for the record.

@dagood dagood merged commit f8acfcc into microsoft/main Mar 19, 2026
43 of 46 checks passed
@dagood dagood deleted the dev/mclayton/fix-pipeline-order branch March 19, 2026 21:10
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.

4 participants