Merge pull request #5 from mineiros-io/mariux/fix-terraform-aws-modul…

…e-validation

fix terraform aws module validation

pre_commit_hooks/terraform/fmt.sh

@@ -1,11 +1,8 @@
 #!/bin/bash
+set -e
 
-# set -e is considered to be a bad practice as written here http://mywiki.wooledge.org/BashFAQ/105
-# so we use a trap instead
-trap 'exit' ERR
-
-# Make environment variables working in OSX GUI apps such as Github Desktop https://stackoverflow.com/q/135688/483528
+# Make environment variables working in OSX GUI apps such as Github Desktop
+# https://stackoverflow.com/q/135688/483528
 export PATH=$PATH:/usr/local/bin
 
 terraform fmt -recursive
-

pre_commit_hooks/terraform/tflint.sh

@@ -1,10 +1,8 @@
 #!/bin/bash
+set -e
 
-# set -e is considered to be a bad practice as written here http://mywiki.wooledge.org/BashFAQ/105
-# so we use a trap instead
-trap 'exit' ERR
-
-# Make environment variables working in OSX GUI apps such as Github Desktop https://stackoverflow.com/q/135688/483528
+# Make environment variables working in OSX GUI apps such as Github Desktop
+# https://stackoverflow.com/q/135688/483528
 export PATH=$PATH:/usr/local/bin
 
 for file in "$@"; do

pre_commit_hooks/terraform/validate.sh

@@ -1,16 +1,14 @@
 #!/bin/bash
+set -e
 
-# set -e is considered to be a bad practice as written here http://mywiki.wooledge.org/BashFAQ/105
-# so we use a trap instead
-trap 'exit' ERR
-
-# Make environment variables working in OSX GUI apps such as Github Desktop https://stackoverflow.com/q/135688/483528
+# Make environment variables working in OSX GUI apps such as Github Desktop
+# https://stackoverflow.com/q/135688/483528
 export PATH=$PATH:/usr/local/bin
 
 # set default aws region for validating aws providers
 export AWS_REGION=${AWS_REGION:="none"}
 
-for dir in $(echo "$@" | xargs -n1 dirname | sort -u | uniq); do
+while read dir; do
   terraform init -backend=false "$dir"
   terraform validate "$dir"
-done
+done < <(printf '%s\n' "${@%/*}" | sort -u)