Re: [DISCUSS] Blocking merge button in PR

2019-10-19 Thread Nabarun Nag
No one felt one review was too much. At least one is bare minimum. Regards Naba On Sat, Oct 19, 2019 at 11:33 AM Owen Nichols wrote: > +1 for starting with only Build, Unit, and Stress tests. > > When you say you propose to "require pull request reviews" before merging, > what number of review

Re: [DISCUSS] Blocking merge button in PR

2019-10-19 Thread Owen Nichols
+1 for starting with only Build, Unit, and Stress tests. When you say you propose to "require pull request reviews" before merging, what number of reviews are you proposing? When we discussed this before