Skip to content

XS✔ ◾ Handle Fork Pull Requests - #847

Closed
Muiris Woulfe (muiriswoulfe) wants to merge 2 commits into
mainfrom
fix/fork-pull-request-safety
Closed

XS✔ ◾ Handle Fork Pull Requests#847
Muiris Woulfe (muiriswoulfe) wants to merge 2 commits into
mainfrom
fix/fork-pull-request-safety

Conversation

@muiriswoulfe

Copy link
Copy Markdown
Member

Purpose

Prevent fork-originated pull requests from running privileged automation when repository credentials are unavailable.

Impact

Fork pull requests retain read-only build and security checks. Privileged updates are skipped, and reviewers receive a caution about reduced automation.

Prevent fork-controlled code from accessing production OIDC
credentials while retaining read-only checks.

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

microsoft-pr-metrics Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

PR Metrics

Thanks for keeping your pull request small.
Thanks for adding tests.

Lines
Product Code -
Test Code -
Subtotal -
Ignored Code 43
Total 43

Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs!

@microsoft-pr-metrics microsoft-pr-metrics Bot changed the title Handle Fork Pull Requests XS✔ ◾ Handle Fork Pull Requests Aug 25, 2026

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

Prevents privileged automation on fork pull requests while retaining safe build and security validation.

Changes:

  • Skips credential-dependent build jobs for fork pull requests.
  • Adds a reviewer caution comment when a fork pull request opens.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/build.yml Gates privileged jobs to trusted branches and events.
.github/workflows/fork-pull-request-caution.yml Posts guidance for reviewing fork contributions.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

The privileged trigger only posts static text and never checks out or
executes pull request code.

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

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@muiriswoulfe
Muiris Woulfe (muiriswoulfe) deleted the fix/fork-pull-request-safety branch August 25, 2026 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants