fix: Speed up x2 TFLint hook execution in dirs with violations (#514)

Co-authored-by: Maksym Vlasov <MaxymVlasov@users.noreply.github.com>
antonbabenko · Apr 28, 2023 · 49974ab · 49974ab
1 parent 0981505
commit 49974ab
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/hooks/terraform_tflint.sh b/hooks/terraform_tflint.sh
@@ -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
 }