Merge pull request #2 from mineiros-io/hooks-should-be-executable-per…

…-default Hooks should be executable per default
mineiros-io · Jan 7, 2020 · 8c274aa · 8c274aa
2 parents 915ee4a + 3a9822d
commit 8c274aa
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 1 deletion.
diff --git a/.pre-commit-hooks.yaml b/.pre-commit-hooks.yaml
@@ -24,7 +24,7 @@
 - id: tflint
   name: tflint
   description: TFLint is a Terraform linter focused on possible errors, best practices, etc. (Terraform >= 0.12)
-  entry: hooks/tflint.sh
+  entry: pre_commit_hooks/tflint.sh
   language: script
   files: \.tf$
   exclude: \.+.terraform\/.*$
diff --git a/README.adoc b/README.adoc
@@ -56,3 +56,12 @@ pre-commit run terraform-validate --all-files
 
 pre-commit run --all-files
 ```
+
+=== Contribute
+
+All `*.sh` files inside pre_commit_hooks need to be marked as executable. Otherwise the pre-commot hooks won't be executable
+by default. You apply the flag to all `*.sh` files with the following command:
+
+```
+find pre_commit_hooks/ -type f -iname "*.sh" -exec chmod +x {} \;
+```
diff --git a/pre_commit_hooks/terraform/fmt.sh b/pre_commit_hooks/terraform/fmt.sh
diff --git a/pre_commit_hooks/terraform/tflint.sh b/pre_commit_hooks/terraform/tflint.sh
diff --git a/pre_commit_hooks/terraform/validate.sh b/pre_commit_hooks/terraform/validate.sh