Skip to content

Commit

Permalink
fix: Speed up x2 TFLint hook execution in dirs with violations (#514)
Browse files Browse the repository at this point in the history
Co-authored-by: Maksym Vlasov <[email protected]>
  • Loading branch information
lexton and MaxymVlasov authored Apr 28, 2023
1 parent 0981505 commit 49974ab
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions hooks/terraform_tflint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -50,16 +50,15 @@ function per_dir_hook_unique_part {
shift
local -a -r args=("$@")

# Print checked PATH **only** if TFLint have any messages
# shellcheck disable=SC2091,SC2068 # Suppress error output
$(tflint ${args[@]} 2>&1) 2> /dev/null || {
common::colorify "yellow" "TFLint in $dir_path/:"
TFLINT_OUTPUT=$(tflint "${args[@]}" 2>&1)
local exit_code=$?

tflint "${args[@]}"
}
if [ $exit_code -ne 0 ]; then
common::colorify "yellow" "TFLint in $dir_path/:"
echo "$TFLINT_OUTPUT"
fi

# return exit code to common::per_dir_hook
local exit_code=$?
return $exit_code
}

Expand Down

0 comments on commit 49974ab

Please sign in to comment.