[Twisted-Python] Enable auto-merge for twisted/twisted

Glyph glyph at twistedmatrix.com
Sat Jul 17 18:02:48 MDT 2021


Nope, I'm fully in favor. When we're done reviewing, and waiting for CI, reviewers should be able to set auto-merge and walk away.  Thank you for raising the issue!

> On Jul 14, 2021, at 5:33 AM, Adi Roiban <adiroiban at gmail.com> wrote:
> 
> Hi,
> 
> Do you see any issues if we enable the auto-merge feature for twisted/twisted?
> 
> The feature is documented here
> 
> https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/automatically-merging-a-pull-request <https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/automatically-merging-a-pull-request>
> 
> With this, in theory you don't have to wait for all the CI to be green before a merge.
> You got your reviews, you fix some typos that don't require a re-review and then set the merge message and your are done.
> Or there was a flaky test failure and just re-run the job and the merge will be done once all the jobs are green again.
> 
> -------------
> 
> This can be abused if someone will enable auto-merge before the PR is approved by a reviewer.
> As most of the time the reviewer will accept the PR but will request or suggest minor changes.
> If the auto-merge is enabled, the PR is mergesd as soon as it is reviewed.
> 
> -------------
> The config looks like this:
> 
> You can allow setting pull requests to merge automatically once all required reviews and status checks have passed.
> 
> Allow auto-merge
> Waits for merge requirements to be met and then merges automatically.
> 
> 
> ------------
> 
> There is also the option to block a merge only after all the conversations are marked as resolved.
> I am using this for my project and it works ok... a bit annoying but can help if you accedentally forgot to address a comment.
> 
> Cheers
> 
> 
> -- 
> Adi Roiban
> _______________________________________________
> Twisted-Python mailing list
> Twisted-Python at twistedmatrix.com
> https://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python

-------------- next part --------------
An HTML attachment was scrubbed...
URL: </pipermail/twisted-python/attachments/20210717/4918778a/attachment.htm>


More information about the Twisted-Python mailing list