<p dir="ltr">As a side note, I think you can force Travis to re run on top of the current master by closing and reopening the issue.</p>
<br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 9, 2015, 10:21 Ryan May <<a href="mailto:rmay31@gmail.com">rmay31@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">We use it here at work. Protected branches are just that--you CANNOT push a commit (i.e. a hash) until it has passed the checks. One note, though, is that the commit must have passed the checks running on top of *current* master. That means if master changes (i.e. you merge another PR), you have to update the PR (you can use the web button to merge master into the branch, but ewww) and re-run the status checks before it can merge.<div><br></div><div>For us, it's still been better to enforce that tests stay green, but we don't have the rate of PR's that matplotlib has.</div><div><br></div><div>Ryan</div></div><div class="gmail_extra"><br><div class="gmail_quote"></div></div><div class="gmail_extra"><div class="gmail_quote">On Fri, Nov 6, 2015 at 10:07 PM, Nathaniel Smith <span dir="ltr"><<a href="mailto:njs@pobox.com" target="_blank">njs@pobox.com</a>></span> wrote:<br></div></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Fri, Nov 6, 2015 at 8:45 PM, Thomas Caswell <<a href="mailto:tcaswell@gmail.com" target="_blank">tcaswell@gmail.com</a>> wrote:<br>
> Hey all,<br>
><br>
> GH now lets you protect branches which prevents people from force-pushing or<br>
> deleting those branches;<br>
> <a href="https://github.com/matplotlib/matplotlib/settings/branches" rel="noreferrer" target="_blank">https://github.com/matplotlib/matplotlib/settings/branches</a>.<br>
><br>
> I have protected our 'production' branches.<br>
><br>
> There is also an option to require status checks to pass before merging to a<br>
> given branch, but that would break our current cherry-pick workflow and mean<br>
> that we could not use our judgement to over-rule <a href="http://travis.ci" rel="noreferrer" target="_blank">travis.ci</a> ;)<br>
<br>
</span>It looks like you can set it so that the status checks are enforced on<br>
normal users, but administrators are allowed to ignore it:<br>
<a href="https://help.github.com/articles/enabling-required-status-checks/" rel="noreferrer" target="_blank">https://help.github.com/articles/enabling-required-status-checks/</a><br>
<br>
What I really want to know is whether it prevents simply pushing<br>
directly to the branch. That would be really awesome because it would<br>
mean you could hand out commit bits like candy without worrying that<br>
unreviewed code could creep into the repo... but there's no indication<br>
of this anywhere. (OTOH if direct pushes *are* still allowed, then<br>
that also provides a method for circumventing the status checks when<br>
desired -- just do a manual merge and push from the command line.)<br>
</blockquote></div></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><font color="#888888"><br>
-n<br>
<br>
--<br>
Nathaniel J. Smith -- <a href="http://vorpus.org" rel="noreferrer" target="_blank">http://vorpus.org</a><br></font></span></blockquote></div></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><font color="#888888">
_______________________________________________<br>
Matplotlib-devel mailing list<br>
<a href="mailto:Matplotlib-devel@python.org" target="_blank">Matplotlib-devel@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/matplotlib-devel" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/matplotlib-devel</a><br>
</font></span></blockquote></div></div><div class="gmail_extra"><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div>Ryan May<br><br></div></div></div>
</div></blockquote></div>