Automatically merge PRs #1802
Replies: 2 comments
-
I would like to do anything we can to improve the speed of PR reviews and merges but I'm hesitant. Bots don't know the difference between a maintainer approval and an approver approval, nor does it know the difference between a large-impact PR and a low-impact PR. We also have the repository setting set to 2 required reviews even though our actual policy is 3. This allows maintainers to merge PRs that are low-risk more quickly. If we enabled this bot, we would need to bump the repository setting to 3. I also think the primary thing holding up most PRs is getting the appropriate number of reviews, not that reviewed PRs are waiting merge. |
Beta Was this translation helpful? Give feedback.
-
Makes sense, thanks! |
Beta Was this translation helpful? Give feedback.
-
https://docs.github.com/en/free-pro-team@latest/github/collaborating-with-issues-and-pull-requests/automatically-merging-a-pull-request
...upon tests passing and the required # of approvals, thoughts?
Beta Was this translation helpful? Give feedback.
All reactions