[Twisted-Python] Fwd: Enable auto-merge for twisted/twisted
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-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
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
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-request... https://docs.github.com/en/github/collaborating-with-issues-and-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@twistedmatrix.com https://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python
On Sun, 18 Jul 2021 at 01:04, Glyph
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!
I have enabled auto-merge for twisted/twisted Let me know if you want to be enabled for other repos. -- Adi Roiban
I think we should enable it everywhere. -g
On Jul 19, 2021, at 1:23 AM, Adi Roiban
wrote: On Sun, 18 Jul 2021 at 01:04, Glyph
mailto:glyph@twistedmatrix.com> wrote: 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! I have enabled auto-merge for twisted/twisted
Let me know if you want to be enabled for other repos.
-- Adi Roiban _______________________________________________ Twisted-Python mailing list Twisted-Python@twistedmatrix.com https://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python
participants (2)
-
Adi Roiban
-
Glyph