On 2015-06-09 20:40:48 +0000 (+0000), Jeremy Stanley wrote: > On 2015-06-09 22:58:15 +0300 (+0300), Kirill Zaitsev wrote: > [...] > > What would you say about adding a job to murano-deployment, that > > would launch > > shellcheck http://www.shellcheck.net/about.html against all .sh > > files? > [...] > > Ooh, that looks neat. Note we also already use > http://git.openstack.org/cgit/openstack-dev/bashate/ to do this for > DevStack and a lot of the Infra repos. Might be worthwhile to get > the rules from one implemented in the other or vice versa.
Er, actually since shellcheck is a static analysis tool and bashate is a style checker, they seem entirely complimentary. Wouldn't make sense to try to incorporate one into the other, but would potentially be useful to run both on proposed changes. Also, since shellcheck (0.3.3) is available in Ubuntu trusty-backports it would be entirely reasonable to install for such a CI job. -- Jeremy Stanley __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev