Skip to content

Remove GitHub build exclusions#549

Merged
MGudgin merged 1 commit into
mainfrom
bbonaby-patch-1
Jun 23, 2026
Merged

Remove GitHub build exclusions#549
MGudgin merged 1 commit into
mainfrom
bbonaby-patch-1

Conversation

@bbonaby

@bbonaby bbonaby commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

📖 Description

We currently exclude .md files from the Build workflow here:

paths-ignore:

However, in the repository settings, the GitHub Actions checks are marked as required. As a result, when a PR only changes .md files, such as #539, the Build workflow does not run because those files are excluded. Since the required check is never produced, GitHub blocks the PR from merging due to a missing required status check. This change now removes the excludes. We will now run the pipeline for all PRs so we can keep the pipelines as required.

🔗 References

🔍 Validation

✅ Checklist

📋 Issue Type

  • Bug fix
  • Feature
  • Task

GitHub Actions runs the PR validation build automatically. The ADO pipeline
(MXC-PR-Build) is the official build pipeline that signs the binaries; it
runs on merge to main and nightly, and Microsoft reviewers can trigger it
on a PR with /azp run. See docs/pull-requests.md.

Microsoft Reviewers: Open in CodeFlow

Remove GitHub build exclusions that prevent PRs that update docs from being checked in due to required pipelines being stuck.
Copilot AI review requested due to automatic review settings June 23, 2026 04:08
@bbonaby bbonaby requested a review from a team as a code owner June 23, 2026 04:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes paths-ignore filters from the GitHub Actions Build workflow so that the required Build check is always generated, even for PRs that only change documentation/markdown or repo metadata files.

Changes:

  • Removed paths-ignore for pull_request so docs-only PRs still trigger the Build workflow (avoids “missing required status check” merge blocks).
  • Removed paths-ignore for push to main so the workflow runs for all pushes as well.

@MGudgin MGudgin merged commit fedb109 into main Jun 23, 2026
19 checks passed
@MGudgin MGudgin deleted the bbonaby-patch-1 branch June 23, 2026 14:43
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