search

compiler-explorer / infra

Infrastructure to set up the public Compiler Explorer instances and compilers
https://godbolt.org
BSD 2-Clause "Simplified" License
344 stars 297 forks source link

Shell script static analysis tool for shell scripts #205

Open Cosaquee opened 5 years ago

Cosaquee commented 5 years ago

As there is quite a lot of shell code in Compiler Explorer it might be worth to incorporate https://github.com/koalaman/shellcheck to be run during CI process.

apmorton commented 3 years ago

It doesn't seem to have made it into CI, but it does look like matt ran shellcheck on our scripts at some point (as I see # shellcheck directives in some places).

We are definitely reducing the amount of shell script over time as well (in favor of our yaml based compiler/library stuff), so this is likely less of a pressing concern as when the issue was originally filed