Add shell script linting: Check for shellcheck warnings in shell scripts

FeiyanLv · Apr 4, 2018 · bc4a099 · bc4a099
1 parent b95d3b8
commit bc4a099
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/Makefile b/Makefile
@@ -263,7 +263,10 @@ check-includes:
 check-cppcheck: .cppcheck-suppress
 	@trap 'rm -f .cppcheck-suppress' 0; git ls-files -- "*.c" "*.h" | grep -vE '^ccan/' | xargs cppcheck -q --language=c --std=c11 --error-exitcode=1 --suppressions-list=.cppcheck-suppress
 
-check-source: check-makefile check-source-bolt check-whitespace check-markdown check-spelling check-python check-includes check-cppcheck
+check-shellcheck:
+	git ls-files -- "*.sh" | xargs shellcheck
+
+check-source: check-makefile check-source-bolt check-whitespace check-markdown check-spelling check-python check-includes check-cppcheck check-shellcheck
 
 full-check: check check-source
 

diff --git a/contrib/Dockerfile.builder b/contrib/Dockerfile.builder
@@ -11,6 +11,7 @@ RUN apt-get -qq update && \
 	automake \
 	clang \
 	cppcheck \
+	shellcheck \
 	eatmydata \
 	software-properties-common \
 	build-essential \

diff --git a/contrib/Dockerfile.builder.i386 b/contrib/Dockerfile.builder.i386
@@ -11,6 +11,7 @@ RUN apt-get -qq update && \
 	automake \
 	clang \
 	cppcheck \
+	shellcheck \
 	eatmydata \
 	software-properties-common \
 	build-essential \