Skip to content

Run the two-human-reviewers gate on pull_request instead of pull_request_target - #315

Merged
Parth Aggarwal (parth21999) merged 2 commits into
masterfrom
paaggarwal/two-human-reviewers-on-pull-request
Oct 23, 2026
Merged

Parth Aggarwal (parth21999) merged 2 commits into
masterfrom
paaggarwal/two-human-reviewers-on-pull-request

Conversation

@parth21999

@parth21999 Parth Aggarwal (parth21999) commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

GitHub starts blocking pull_request_target by default in public repositories on November 2, 2026 (default policy). This updates .github/workflows/two-human-reviewers.yml to the new caller template from Azure/c-build-tools#475, and doesn't need that PR to merge first.

  • The gate triggers on pull_request instead of pull_request_target, with the same activity types. The pull_request_review and workflow_dispatch triggers, the permissions and the mrbot/two-human-reviewers status are unchanged.
  • The header comment is the template's new, shorter one.
  • The reusable workflow pin moves from ea713a3 to e5ade18, this repository's deps/c-build-tools commit. The workflow file is identical at both.

Fork PRs: a fork's pull_request run gets a read-only token, so it can't publish the status, and the PR stays blocked until a maintainer publishes it with Run workflow on the Actions tab, or:

gh workflow run two-human-reviewers.yml -R Azure/com-wrapper -f pr=<number>

Run it again after each push.

Validation: this PR's own pull_request run published mrbot/two-human-reviewers = success, with no publish warning.

…est_target

GitHub starts blocking pull_request_target by default in public repositories on November 2, 2026.
Update the caller to the new template from Azure/c-build-tools#475.

Pin the reusable workflow to e5ade18, this repository's deps/c-build-tools commit. The workflow
file is identical at ea713a3, the previous pin.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e7c537a1-1b1b-413e-9943-8d3174d1c5ce
@parth21999

Copy link
Copy Markdown
Member Author

/azp run com-wrapper-gate

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@azure-pipelines

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

@parth21999

Copy link
Copy Markdown
Member Author

/azp run com-wrapper-gate

@azure-pipelines

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

@parth21999
Parth Aggarwal (parth21999) merged commit 190cab0 into master Oct 23, 2026
19 checks passed
@parth21999
Parth Aggarwal (parth21999) deleted the paaggarwal/two-human-reviewers-on-pull-request branch October 23, 2026 03:12
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.

2 participants