Skip to content

mu-pr-validation-pending.yml: Temporarily disable commit status updates#870

Merged
makubacki merged 1 commit intomicrosoft:release/202511from
makubacki:temp_disable_pr_val_commit_status
Apr 8, 2026
Merged

mu-pr-validation-pending.yml: Temporarily disable commit status updates#870
makubacki merged 1 commit intomicrosoft:release/202511from
makubacki:temp_disable_pr_val_commit_status

Conversation

@makubacki
Copy link
Copy Markdown
Member

Description

The Mu Automation GitHub app requires a permission update to be accepted by a microsoft org admin. Until that happens, disable the calling the GitHub API to update the commit status.

This will still run validation and post PR comments. The pass/fail result just won't be reported in the status check area of the PR until the permission is allowed.

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

How This Was Tested

  • N/A

Integration Instructions

  • N/A

The Mu Automation GitHub app requires a permission update to be
accepted by a microsoft org admin. Until that happens, disable the
calling the GitHub API to update the commit status.

This will still run validation and post PR comments. The pass/fail
result just won't be reported in the status check area of the PR
until the permission is allowed.

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
@makubacki makubacki requested review from Javagedes, apop5 and eeshanl April 8, 2026 20:09
@makubacki makubacki self-assigned this Apr 8, 2026
@makubacki makubacki marked this pull request as ready for review April 8, 2026 20:09
@mu-automation
Copy link
Copy Markdown
Contributor

mu-automation Bot commented Apr 8, 2026

✅ QEMU Validation Passed

Source Dependencies

Repository Commit
mu_plus 787352d
mu_tiano_platforms c1a6ff0

Results

Platform Target Build Boot Overall Boot Time Build Logs Boot Logs
Q35 DEBUG ✅ success ✅ success 0m 17s Build Logs Boot Logs
SBSA DEBUG ✅ success ✅ success 0m 30s Build Logs Boot Logs

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

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

@mu-automation mu-automation Bot added the impact:non-functional Does not have a functional impact label Apr 8, 2026
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 8, 2026

Codecov Report

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

Additional details and impacted files
@@                Coverage Diff                @@
##             release/202511     #870   +/-   ##
=================================================
  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.

@makubacki makubacki changed the title mu-pr-validation-pending.yml: Temporarily disable commit status upates mu-pr-validation-pending.yml: Temporarily disable commit status updates Apr 8, 2026
@makubacki makubacki merged commit f3a8e88 into microsoft:release/202511 Apr 8, 2026
31 of 32 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