Merge pull request #47 from cyberark/shellcheck_opts

Pass SHELLCHECK_OPTS to Shell check
cyberark · Aug 23, 2021 · a0644c1 · a0644c1
2 parents 09b7000 + c8270ad
commit a0644c1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.0.6] - 2021-08-23
+### Changed
+- bl_shellcheck_script now passes SHELLCHECK_OPTS through to the container that runs shellcheck
+
 ## [2.0.5] - 2021-02-16
 ### Changed
 - bl_all_files_in_repo now excludes submodules

diff --git a/test-utils/lib b/test-utils/lib
@@ -24,7 +24,11 @@ function bl_shellcheck_script(){
 
     if bash -n "${script}"; then
         # shellcheck disable=SC2086
-        docker run --rm -v "${PWD}:/mnt" ${SHELLCHECK_IMAGE}:${SHELLCHECK_TAG} ${ignores} ${script}
+        docker run \
+            --rm \
+            -v "${PWD}:/mnt" \
+            -e SHELLCHECK_OPTS \
+            ${SHELLCHECK_IMAGE}:${SHELLCHECK_TAG} ${ignores} ${script}
     else
         return 1
     fi