search

dedis / cothority

Scalable collective authority
Other
426 stars 106 forks source link

Use linter for shell scripts #972

Open kc1212 opened 6 years ago

kc1212 commented 6 years ago

Twin of https://github.com/dedis/onet/issues/269

ValentinMoullet commented 6 years ago

From the ShellCheck Github page: "Travis CI has now integrated ShellCheck by default, so you don't need to manually install it."

But I wasn't sure how we should integrate the linting of Go files with Bash files; do we need another different .travis.yml only for bash files? Also we need to add it to when we make test.

jeffallen commented 6 years ago

Start with adding it as a target to the Makefile, because adding a whole new dimension to the travis matrix is way too heavy-weight.