Skip to content

Answer the build review gate only from reviewers who can write to the repo - #832

Open
czpython wants to merge 1 commit into
mainfrom
review-gate-writers-only
Open

czpython wants to merge 1 commit into
mainfrom
review-gate-writers-only

Conversation

@czpython

@czpython czpython commented Oct 5, 2026

Copy link
Copy Markdown
Owner

Any GitHub account can submit an APPROVE review on a public repo. Before this change, such a review answered the Software Factory build's ReviewWork gate. With the default on_approval: merge, the build then merged the PR.

  • pr.review_submitted now carries author_can_write, from the review's author_association. Comments already carry the same fact.
  • pr_review_answers_the_gate subscribes with payload__author_can_write=True. A review from an account that cannot write to the repo leaves the gate parked.

This branch has not been deployed

No deployments
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.

1 participant