gow icon indicating copy to clipboard operation
gow copied to clipboard

Add an action to lint shell scripts and Dockerfiles

Open zb140 opened this issue 2 years ago • 4 comments

Added an action to run linters for shell scripts and Dockerfiles. It will run on any PR. If there's a lint issue in one of the files being modified, it will add a review pointing out the specific line and error. If it finds errors in files not in the PR, those will get noted under the "Checks" tab. None of this blocks merging the PR anyway, but it should make it much easier to spot potential problems.

zb140 avatar Sep 18 '22 15:09 zb140

Note that I have not yet gone through and fixed all the existing lint issues; my plan is to do that after merging #76, since there's changes to pretty much all the files that would be affected by this.

zb140 avatar Sep 18 '22 16:09 zb140

While we're at it...

https://github.com/dotenv-linter/action-dotenv-linter

https://github.com/marketplace/actions/yaml-lint

Sparticuz avatar Sep 20 '22 15:09 Sparticuz

While we're at it...

https://github.com/dotenv-linter/action-dotenv-linter

https://github.com/marketplace/actions/yaml-lint

Good idea! I've added them.

zb140 avatar Sep 24 '22 16:09 zb140

Lgtm

Sparticuz avatar Oct 12 '22 10:10 Sparticuz