Skip to content

Commit c8270ad

Browse files
committed
Pass SHELLCHECK_OPTS to Shell check
This allows users of bl_shellcheck_script to specifiy their own configuration options (eg ignores) but still use bl_shellcheck_script. Related: conjurinc/ops#753
1 parent 09b7000 commit c8270ad

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
66

77
## [Unreleased]
88

9+
## [2.0.6] - 2021-08-23
10+
### Changed
11+
- bl_shellcheck_script now passes SHELLCHECK_OPTS through to the container that runs shellcheck
12+
913
## [2.0.5] - 2021-02-16
1014
### Changed
1115
- bl_all_files_in_repo now excludes submodules

test-utils/lib

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,11 @@ function bl_shellcheck_script(){
2424

2525
if bash -n "${script}"; then
2626
# shellcheck disable=SC2086
27-
docker run --rm -v "${PWD}:/mnt" ${SHELLCHECK_IMAGE}:${SHELLCHECK_TAG} ${ignores} ${script}
27+
docker run \
28+
--rm \
29+
-v "${PWD}:/mnt" \
30+
-e SHELLCHECK_OPTS \
31+
${SHELLCHECK_IMAGE}:${SHELLCHECK_TAG} ${ignores} ${script}
2832
else
2933
return 1
3034
fi

0 commit comments

Comments
 (0)