update shellcheck ignores

This commit is contained in:
Jose Diaz-Gonzalez
2016-05-08 15:00:09 -04:00
parent b448a83c1e
commit 2d7a1ab5b2

View File

@@ -24,11 +24,10 @@ ci-dependencies: shellcheck bats
lint:
# these are disabled due to their expansive existence in the codebase. we should clean it up though
# SC2046: Quote this to prevent word splitting. - https://github.com/koalaman/shellcheck/wiki/SC2046
# SC2068: Double quote array expansions, otherwise they're like $* and break on spaces. - https://github.com/koalaman/shellcheck/wiki/SC2068
# SC2086: Double quote to prevent globbing and word splitting - https://github.com/koalaman/shellcheck/wiki/SC2086
# SC1090: Can't follow non-constant source. Use a directive to specify location.
# SC2155: Declare and assign separately to avoid masking return values.
@echo linting...
@$(QUIET) find ./ -maxdepth 1 -not -path '*/\.*' | xargs file | egrep "shell|bash" | awk '{ print $$1 }' | sed 's/://g' | xargs shellcheck -e SC2046,SC2068,SC2086
@$(QUIET) find ./ -maxdepth 1 -not -path '*/\.*' | xargs file | egrep "shell|bash" | awk '{ print $$1 }' | sed 's/://g' | xargs shellcheck -e SC1090,SC2155
unit-tests:
@echo running unit tests...