Skip to content

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

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

Parth Aggarwal (parth21999) merged 1 commit 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.

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/macro-utils-c -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.

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 Azure-Macro-Utils-C-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
Parth Aggarwal (parth21999) merged commit 3cda4c4 into master Oct 23, 2026
18 checks passed
@parth21999
Parth Aggarwal (parth21999) deleted the paaggarwal/two-human-reviewers-on-pull-request branch October 23, 2026 03:14
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