<div dir="ltr">So this will require bash scripts to adhere to bashate before being accepted? Is it possible to have the check as non-voting? Does this open the door to having other file types be checked?<div><br></div><div>IMHO, it's more important for the OSOps project to foster collaboration and contributions rather than worry about an accepted style.<div><br></div><div>As an example, yesterday's commits used hard-tabs:</div><div><br></div><div><a href="https://review.openstack.org/#/c/228545/">https://review.openstack.org/#/c/228545/</a><br></div><div><a href="https://review.openstack.org/#/c/228534/">https://review.openstack.org/#/c/228534/</a><br></div><div><br></div><div>I think we're going to see a lot of variation of styles coming in.</div><div><br></div><div>I don't want to come off as sounding ignorant or disrespectful to other projects that have guidelines in place -- I fully understand and respect those decisions.<br></div><div><br></div><div>Joe</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 29, 2015 at 12:52 PM, JJ Asghar <span dir="ltr"><<a href="mailto:jj@chef.io" target="_blank">jj@chef.io</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Awesome! That works!<br>
<span class="im HOEnZb"><br>
Best Regards,<br>
JJ Asghar<br>
c: <a href="tel:512.619.0722" value="+15126190722">512.619.0722</a> t: @jjasghar irc: j^2<br>
<br>
</span><span class="im HOEnZb">On 9/29/15 1:27 PM, Christian Berendt wrote:<br>
> On 09/29/2015 07:45 PM, JJ Asghar wrote:<br>
>> So this popped up today[1]. This seems like something that should be<br>
>> leveraged in our gates/validations?<br>
><br>
> I prepared review requests to enable checks on the gates for<br>
><br>
> * osops-tools-monitoring: <a href="https://review.openstack.org/#/c/229094/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/229094/</a><br>
> * osops-tools-generic: <a href="https://review.openstack.org/#/c/229043/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/229043/</a><br>
><br>
> Christian.<br>
><br>
<br>
<br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
OpenStack-operators mailing list<br>
<a href="mailto:OpenStack-operators@lists.openstack.org">OpenStack-operators@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-operators" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-operators</a><br>
</div></div></blockquote></div><br></div>