Skip to content
Permalink
Browse files

DO NOT MERGE

attempting to debug paths, making shellcheck run first in CI
  • Loading branch information...
BenTheElder committed Jan 11, 2019
1 parent d150df8 commit dab479478e2908ac5ca6b2ee6529b005677df549
Showing with 8 additions and 1 deletion.
  1. +8 −1 hack/{verify-shellcheck.sh → verify-aa-shellcheck.sh}
@@ -18,6 +18,8 @@ set -o errexit
set -o nounset
set -o pipefail

set -x;

KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/..
source "${KUBE_ROOT}/hack/lib/init.sh"

@@ -121,7 +123,12 @@ errors=()
not_failing=()
for f in "${all_shell_scripts[@]}"; do
set +o errexit
failedLint=$(docker exec -t "${SHELLCHECK_CONTAINER}" shellcheck --exclude="${SHELLCHECK_DISABLED}" "${f}")
failedLint=$(docker exec \
-t \
-w "${KUBE_ROOT}" \
"${SHELLCHECK_CONTAINER}" \
shellcheck --exclude="${SHELLCHECK_DISABLED}" "${f}" \
)
set -o errexit
array_contains "${f}" "${failing_files[@]}" && in_failing=$? || in_failing=$?
if [[ -n "${failedLint}" ]] && [[ "${in_failing}" -ne "0" ]]; then

0 comments on commit dab4794

Please sign in to comment.
You can’t perform that action at this time.