<div dir="ltr">Hi,<div><br></div><div>Seems like all protections (against force push, deleting and failing tests) are a good fit.<br><div class="gmail_extra"><br></div><div class="gmail_extra">There may be instances where some tests are failing on Mac, due to an oddity with that platform and not with the actual code per se, i.e. the test needs updated for Mac or some Mac-specific bit needs added to source. In light of such cases, it might be best to not require that all tests pass for the Merge button to be enabled, or it may force some devs to wait until Mac-specific issues are addressed first.</div><div class="gmail_extra"><br></div><div class="gmail_extra"><div><div class="gmail_signature">If we if strictly enforce all tests pass, maybe there is a way to tell Travis that a failure on one of the test platforms is non-critical (Mac), but a failure on another (Linux) is. ??</div><div class="gmail_signature"><br></div><div class="gmail_signature">Regards,</div><div class="gmail_signature"><br>Larry Shaffer<br>Dakota Cartography<br>Black Hills, South Dakota</div></div>
<br><div class="gmail_quote">On Tue, Oct 13, 2015 at 4:41 PM, Nathan Woodrow <span dir="ltr"><<a href="mailto:madmanwoo@gmail.com" target="_blank">madmanwoo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">+1 not allowing force push is a good thing IMO</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 14, 2015 at 8:39 AM, Nyall Dawson <span dir="ltr"><<a href="mailto:nyall.dawson@gmail.com" target="_blank">nyall.dawson@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This seems like a really good idea -<br>
<a href="https://github.com/blog/2051-protected-branches-and-required-status-checks" rel="noreferrer" target="_blank">https://github.com/blog/2051-protected-branches-and-required-status-checks</a><br>
<br>
I noticed it's now been rolled out, and think we should enable this on<br>
the QGIS repo ASAP. I can't see any downsides to enabling this and<br>
lots of upsides. Can someone with rights do this?<br>
<br>
Nyall<br>
_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a></blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br></blockquote></div><br></div></div></div>