<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 22, 2017 at 8:51 AM, Nick Coghlan <span dir="ltr"><<a href="mailto:ncoghlan@gmail.com" target="_blank">ncoghlan@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 22 February 2017 at 13:12, Nick Coghlan <<a href="mailto:ncoghlan@gmail.com">ncoghlan@gmail.com</a>> wrote:<br>
> I'm +1 for turning on required status checks though - giving us a<br>
> strong incentive to get the test suite stable and keep it that way is<br>
> an unequivocally good thing, and I do want to keep the "PR required,<br>
> even for core developers" experiment going for at least another few<br>
> weeks.<br>
<br>
</span>To be clear: my preference would be to have the setup be "Review<br>
required, but self-review is permitted", as that lets us decided<br>
whether or not it makes sense to wait for the CI to run.<br></blockquote><div><br></div><div>If I'm correct, gh will grey-out (but completly block) merge when all the checks<br></div><div>are not green. Green CI should be helpful here. Eventually we could add some<br></div><div>additional checks for self-approval awareness in our bot. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
But if GitHub doesn't allow that, then I think "Successful CI run<br>
required" would be a better near term gate than preventing developers<br>
from merging our own changes (even when we're importing someone else's<br>
patch from <a href="http://bugs.python.org" rel="noreferrer" target="_blank">bugs.python.org</a> or just fixing a typo in the docs)<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote></div></div></div>