Skip to content

Add PR doc to explain what Microsoft employees need to do to run PR builds#452

Merged
bbonaby merged 2 commits into
mainfrom
user/bbonaby/add-pr-info-doc
May 28, 2026
Merged

Add PR doc to explain what Microsoft employees need to do to run PR builds#452
bbonaby merged 2 commits into
mainfrom
user/bbonaby/add-pr-info-doc

Conversation

@bbonaby

@bbonaby bbonaby commented May 28, 2026

Copy link
Copy Markdown
Collaborator

📖 Description

Due to Microsoft ADO policy auto running PR pipelines is not allowed. A Microsoft employee is required to comment and run the build. This prevents forks from non-Microsoft folks from auto running PR builds with malicious content.

This PR adds a pull request doc to explain how Microsoft employees can start PR builds. This is the same as other repositories that use ADO pipelines like WinAppSDK: E.g microsoft/WindowsAppSDK#6474 for example.

TLDR: Microsoft employees only need to comment on a PR with /azp run and the azure pipeline bot will start the PR pipeline.

🔗 References

🔍 Validation

✅ Checklist

📋 Issue Type

  • Bug fix
  • Feature
  • Task
Microsoft Reviewers: Open in CodeFlow

Copilot AI review requested due to automatic review settings May 28, 2026 22:16
@bbonaby

bbonaby commented May 28, 2026

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

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

Adds a short documentation file explaining how Microsoft employees can trigger the ADO PR build pipeline by commenting /azp run, since Microsoft ADO policy disallows auto-running PR pipelines.

Changes:

  • New doc file docs/pull-requests.md describing the /azp run comment workflow and the MXC-PR-Build pipeline name/link.

- docs/pull-requests.md: lead with the policy reason (prevents external forks from auto-running internal pipeline agents) so the manual /azp run is clearly mandatory, not a workflow quirk.
- PR template: short footer for Microsoft reviewers pointing at the doc.
- CONTRIBUTING.md: same pointer in the Code review section.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bbonaby

bbonaby commented May 28, 2026

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@bbonaby
bbonaby merged commit b722f6e into main May 28, 2026
20 checks passed
@bbonaby
bbonaby deleted the user/bbonaby/add-pr-info-doc branch May 28, 2026 23:27
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