Contributors mailing list archives
Re: New Github Review Optionsby
LasLabs, Dave Lasley
I noticed CI only fails when the review comes from a PSC. Is it so?
We can’t configure the amount of reviews, it is static one per PR. This seems that this fits our rule of requiring one :+1 w/ write access to the repo though, so I think we’re good.
One catch is if someone blocks, they have to unblock or an admin has to override. IMO this is also fine, we ping each other on our blockers anyways - this is just more formal & less likely to be missed.
Dave Lasley - LasLabs
Founder / CEO
<img height="49" width="50" apple-inline="yes" id="BB58E745-D121-4CFC-8F0C-D6DE7425745F" apple-width="yes" apple-height="yes" src="cid:B6781F39-FEE7-4B4E-91AD-FDB83E4A18E3@dlasley.net" class="">
On Sep 19, 2016, at 3:38 AM, Jairo Llopis <firstname.lastname@example.org> wrote:2016-09-17 0:52 GMT+02:00 Eric Caudal <email@example.com>:I propose that we implement these rules (2 approvals required) on OCA repos so that the review approval process is part of our standard CI spot.I noticed CI only fails when the review comes from a PSC. Is it so?