Hi, It looks like we have reached consensus here. I am starting to request infra to enable the "Require pull request reviews before merging" feature and set number to 1.
On Wed, Mar 20, 2019 at 10:45 PM Huxing Zhang <[email protected]> wrote: > > Hi, > > On Wed, Mar 20, 2019 at 6:08 PM Mark Thomas <[email protected]> wrote: > > > > I recommend that you enforce this socially rather than technically. > > > > There are times when a committer needs to get something committed (CI is > > broken, security fix, correct an obvious bug that is blocking something > > else, etc.) and having the flexibility to just commit it can be very > > helpful. > > I think the discussion here does not prevent some one to commit > directly. It is just about if a pull request is raised, it must be > approved. > In some critical cases, one can approve the pull request by > himself/herself with proper comments or commit directly. > > > > > Mark > > > > > > On 20/03/2019 06:37, yuhang xiu wrote: > > > agree. I think this is a very good way to help us standardize the pr > > > merge. > > > > > > YunKun Huang <[email protected]> 于2019年3月20日周三 上午11:41写道： > > > > > >> 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 > > >>> > > >> > > > > > > > > -- > Best Regards！ > Huxing -- Best Regards！ Huxing
