<div dir="ltr">I'm all for style checking.  We may also want to include some syntax checking as well (bash -n <script>).  I personally believe that, so long as the acceptance tests aren't overly strict, they provide a nice guide.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 29, 2015 at 12:43 PM, Joe Topjian <span dir="ltr"><<a href="mailto:joe@topjian.net" target="_blank">joe@topjian.net</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">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/" target="_blank">https://review.openstack.org/#/c/228545/</a><br></div><div><a href="https://review.openstack.org/#/c/228534/" target="_blank">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.<span class="HOEnZb"><font color="#888888"><br></font></span></div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Joe</div></font></span></div></div><div class="HOEnZb"><div class="h5"><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><br>
Best Regards,<br>
JJ Asghar<br>
c: <a href="tel:512.619.0722" value="+15126190722" target="_blank">512.619.0722</a> t: @jjasghar irc: j^2<br>
<br>
</span><span>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><div>_______________________________________________<br>
OpenStack-operators mailing list<br>
<a href="mailto:OpenStack-operators@lists.openstack.org" target="_blank">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>
</div></div><br>_______________________________________________<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>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>David W.</div>Unix, because every barista in Seattle has an MCSE.</div></div>
</div>