Agree, we should have at least one approval before merging
On 2019/03/20 03:00:51, Huxing Zhang <[email protected]> wrote: > Hi, > > > On Wed, Mar 20, 2019 at 10:55 AM YunKun Huang <[email protected]> wrote: > > > > > > I guess you are talking about "Require pull request reviews before merging" > > feature [1] > > Cool! That is what I want. > I am +1 to support enabling this and set the number of required approval to 1. > How do others think? > > > > > [1]https://help.github.com/en/articles/enabling-required-reviews-for-pull-requests > > > > On 2019/03/20 02:39:43, Huxing Zhang <[email protected]> wrote: > > > Hi, > > > > > > This pull request[1] is merged without any comments, or following any > > > review process. > > > Any ideas how to avoid this? > > > Can Github support the feature that disable the merge button until a > > > committer has approved the pull request? > > > > > > [1] https://github.com/apache/incubator-dubbo/pull/3693 > > > -- > > > Best Regards！ > > > Huxing > > > > > > > -- > Best Regards！ > Huxing >
