<div dir="ltr">I think it’s a good idea.  I think scripts and such that don’t pass the listing tools can go into the contrib repos and if they get cleaned up then they can move over to the regular ones.  <div><br></div><div>I don’t actually like some of the PEP8 and bashate rules, but I’d rather have a consistent style than have them in everyone’s personal style.  I find less opinionated tools like rubocop to be less useful, since you end up bogged down in arguing about which style options to choose.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 16, 2015 at 7:03 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">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA512<br>
<br>
NOTE: I get what I'm about to propose will open a HUGE can of worms, but<br>
we need it, so I'll start the conversation.<br>
<br>
We had some initial discussion and thoughts on coding standards when we<br>
first started this project. It got shot down, but not before    Christian<br>
Berendt came up with review[1].<br>
<br>
If you don't know, bashate[2] is a pep8 coding standard for bash. I<br>
think it's a great starting point for our tools-generic repo.<br>
<br>
Are there any objections or concerns with this? If there isn't any,<br>
voiced in a timely fashion, I'll take the action item of getting the<br>
review re-enabled.<br>
<br>
To the can o'worms: I think we should probably set up linters for any<br>
python, ruby, bash, etc scripts also. It's hard to cargo cult these from<br>
other projects in the OpenStack ecosystem; which I understand can be<br>
different per different project. This can be a religious debate, and I<br>
want to acknowledge that people feel strongly about this; but we all<br>
need to come together to get this project off the ground. I'd like to go<br>
as _simple_ as possible, then we can add more opinionated styles, if<br>
desired, as our project grows.<br>
<br>
I propose people post different standards here[3] and when we have our<br>
next meeting we vote to figure out what we should have. If you can't<br>
attend the meeting please put +1s by what you'd like and we can take<br>
that in for consideration.<br>
<br>
Please if you have any concerns, questions, thoughts don't hesitate to<br>
reach out.<br>
<br>
<br>
[1]: <a href="https://review.openstack.org/#/c/229029" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/229029</a><br>
[2]: <a href="https://github.com/openstack-dev/bashate" rel="noreferrer" target="_blank">https://github.com/openstack-dev/bashate</a><br>
[3]: <a href="https://etherpad.openstack.org/p/osops-coding-standards" rel="noreferrer" target="_blank">https://etherpad.openstack.org/p/osops-coding-standards</a><br>
<br>
<br>
- --<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>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG/MacGPG2 v2<br>
Comment: GPGTools - <a href="https://gpgtools.org" rel="noreferrer" target="_blank">https://gpgtools.org</a><br>
<br>
iQIcBAEBCgAGBQJWSm7rAAoJEDZbxzMH0+jTJDYQAJ4dNZQhkH3jmLLBo7iqNXYN<br>
gUngDUHfRL0UY63dC/3v4V7OtH+Eg5QNPXSYvw1CzLZm+KfvfaYTb4PIY9ZAt83f<br>
7h6dCgtF5XESHFHVNoEOAQYpTRM+99H0N0R5DE1alGow+5f2Rj9dWpBD59RYQbFR<br>
TX8IPvEPQRtAvfErt0/O6TxOLR64CSehYRyki3CWgVtBd1Q+cOl+0NV4+cneIqUk<br>
EBEzQglCxtRAb0yWcMVRp5kHzLOB2o+6xzb8PXKYyKr9J1j1e0W679A6qzE3Ut+w<br>
l69mXQxfz8x/0B9lW0xqipgYN+17mxdAnnu3Rhwr85HAMeI5O+M9w0CF7rxrwWII<br>
qZVb8/zZKfX3jK6hga6+mQZgZUAvmxGxV498TEkOZLIuqo+catH7SxADmwpafSuP<br>
J+TrAT97gZrKaGsBCS8Iwzs/4eBJUrL3enX0PXR+UcsCXAo5TjYYx34jwtqviNkI<br>
GNahmLuUoOsszg1WtNKFSs6Cc86u0GPoUer3VmUEwaVkxwyoIpYC5ePmaZZ6MFZb<br>
se75w5t0BUzO4StiJPV7C/U40SuA3DFG308DK1of/isOXJ6kYcraZOMutM+4xznZ<br>
UfCxVF65Tsq6+YRc6XCS5qBoEHGJkm/8iQiy88LlI5RYUcdBFnEBgxggnJWrWPIK<br>
Lsg7ynN9YPQ/0jaWfKxQ<br>
=RLkF<br>
-----END PGP SIGNATURE-----<br>
<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>
</blockquote></div><br></div>