Skip to content

Require two human reviewers on bot-authored pull requests - #514

Merged
Matt Durak (mattdurak) merged 5 commits into
masterfrom
mdurak/mrbot/two-human-reviewers
Aug 5, 2026
Merged

Require two human reviewers on bot-authored pull requests#514
Matt Durak (mattdurak) merged 5 commits into
masterfrom
mdurak/mrbot/two-human-reviewers

Conversation

@mattdurak

Copy link
Copy Markdown
Contributor

A pull request opened by the agent is missing the human who would normally have written and vetted the change, so a single approval would be the only human judgement applied to it. This job requires two distinct human approvals on those PRs and passes immediately on everyone else's, leaving normal review unchanged.

@azure-pipelines

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

The caller pointed at the branch the reusable workflow was developed on, which is deleted once that pull request merges and would take this gate down with it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

:shipit:

@mattdurak

Copy link
Copy Markdown
Contributor Author

/azp run Azure-umock-c-Gate

@azure-pipelines

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

@jebrando Jelani Brandon (jebrando) 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.

:shipit:

@mattdurak
Matt Durak (mattdurak) enabled auto-merge (squash) August 5, 2026 17:29
@mattdurak

Copy link
Copy Markdown
Contributor Author

/azp run Azure-umock-c-Gate

@azure-pipelines

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

@mattdurak
Matt Durak (mattdurak) merged commit 01faf3a into master Aug 5, 2026
16 checks passed
@mattdurak
Matt Durak (mattdurak) deleted the mdurak/mrbot/two-human-reviewers branch August 5, 2026 17:39
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