<div dir="ltr">+1<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 10, 2018 at 3:30 PM, David Davis <span dir="ltr"><<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@redhat.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">We noticed in Pulp that the 2-master branch has branch protection but only to prevent force pushes and deletion. I was wondering if we should also add these checks:<div><br></div><div>- Require an approving review</div><div>- Require status checks (e.g. unit tests, docs test, flake8)<div><br></div><div>If so, I think we should also do this for all master and 2-master branches for all Pulp core repos (where applicable). Does anyone have any thoughts or objections?</div><div><br></div><div>I’ll leave this discussion open until July 22, 2018.</div><span class="HOEnZb"><font color="#888888"><div><div><div dir="ltr" class="m_8368328482659023810gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div></div></font></span></div></div>
<br>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/pulp-dev</a><br>
<br></blockquote></div><br></div>