Skip to content

Skip Mu PR validation for PRs targeting non-default branches#872

Merged
makubacki merged 1 commit intomicrosoft:release/202511from
makubacki:skip_mu_pr_val_targeting_no_default_branches
Apr 9, 2026
Merged

Skip Mu PR validation for PRs targeting non-default branches#872
makubacki merged 1 commit intomicrosoft:release/202511from
makubacki:skip_mu_pr_val_targeting_no_default_branches

Conversation

@makubacki
Copy link
Copy Markdown
Member

Description

Because mu-pr-validation.yml is in the default branch (release/202511) and runs on a workflow_run trigger for the CodeQL workflow where the CodeQL workflow runs for all release branches, the Mu PR Validation workflow will run for all PRs targeting any release branch.

Because mu_tiano_platforms is only compatible with default branches, we want to skip Mu PR Validation targeting those non-default branches.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

Verified for PRs against the default on non-default branch on my fork.

Non-Default Case (Skip)

image

Default Case (Run)

image

Integration Instructions

  • N/A

@makubacki makubacki self-assigned this Apr 9, 2026
@mu-automation
Copy link
Copy Markdown
Contributor

mu-automation bot commented Apr 9, 2026

✅ QEMU Validation Passed

Source Dependencies

Repository Commit
mu_plus c260f31
mu_tiano_platforms e7f5cec

Results

Platform Target Build Boot Overall Boot Time Build Logs Boot Logs
Q35 DEBUG ✅ success ✅ success 0m 16s Build Logs Boot Logs
SBSA DEBUG ✅ success ✅ success 0m 28s Build Logs Boot Logs

Workflow run: https://github.com/microsoft/mu_plus/actions/runs/24196246991

This comment was automatically generated by the Mu QEMU PR Validation workflow.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (release/202511@fb02a76). Learn more about missing BASE report.

Additional details and impacted files
@@                Coverage Diff                @@
##             release/202511     #872   +/-   ##
=================================================
  Coverage                  ?   10.10%           
=================================================
  Files                     ?      149           
  Lines                     ?    24833           
  Branches                  ?     2346           
=================================================
  Hits                      ?     2509           
  Misses                    ?    22288           
  Partials                  ?       36           
Flag Coverage Δ
AdvLoggerPkg 2.86% <ø> (?)
HidPkg 2.79% <ø> (?)
MfciPkg 34.00% <ø> (?)
MsCorePkg 1.30% <ø> (?)
MsWheaPkg 7.37% <ø> (?)
XmlSupportPkg 25.46% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Because mu-pr-validation.yml is in the default branch (release/202511)
and runs on a workflow_run trigger for the CodeQL workflow where the
CodeQL workflow runs for all release branches, the Mu PR Validation
workflow will run for all PRs targeting any release branch.

Because mu_tiano_platforms is only compatible with default branches,
we want to skip Mu PR Validation targeting those non-default branches.

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
@makubacki makubacki force-pushed the skip_mu_pr_val_targeting_no_default_branches branch from c2814ad to c260f31 Compare April 9, 2026 14:32
@makubacki makubacki enabled auto-merge (squash) April 9, 2026 14:36
@makubacki makubacki merged commit 090bb16 into microsoft:release/202511 Apr 9, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact:non-functional Does not have a functional impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants