Merge pull request #16 from Tensho/tflint

Add "tflint" Hook

Author: yorinasub17

.pre-commit-hooks.yaml

@@ -18,6 +18,14 @@
   exclude: \.+.terraform\/.*$
   require_serial: true
 
+- id: tflint
+  name: tflint
+  description: Linter for Terraform source code
+  entry: hooks/tflint.sh
+  language: script
+  files: \.tf$
+  exclude: \.+.terraform\/.*$
+
 - id: shellcheck
   name: Shellcheck Bash Linter
   description: Performs linting on bash scripts

README.md

@@ -7,6 +7,7 @@ supported hooks are:
 
 * **terraform-fmt**: Automatically run `terraform fmt` on all Terraform code (`*.tf` files).
 * **terraform-validate**: Automatically run `terraform validate` on all Terraform code (`*.tf` files).
+* **tflint**: Automatically run [`tflint`](https://github.com/terraform-linters/tflint) on all Terraform code (`*.tf` files).
 * **shellcheck**: Run [`shellcheck`](https://www.shellcheck.net/) to lint files that contain a bash [shebang](https://en.wikipedia.org/wiki/Shebang_(Unix))
 * **gofmt**: Automatically run `gofmt` on all Golang code (`*.go` files).
 * **golint**: Automatically run `golint` on all Golang code (`*.go` files)
@@ -28,6 +29,7 @@ repos:
     hooks:
       - id: terraform-fmt
       - id: terraform-validate
+      - id: tflint
       - id: shellcheck
       - id: gofmt
       - id: golint

hooks/tflint.sh

@@ -0,0 +1,12 @@
+#!/bin/bash
+
+set -e
+
+# OSX GUI apps do not pick up environment variables the same way as Terminal apps and there are no easy solutions,
+# especially as Apple changes the GUI app behavior every release (see https://stackoverflow.com/q/135688/483528). As a
+# workaround to allow GitHub Desktop to work, add this (hopefully harmless) setting here.
+export PATH=$PATH:/usr/local/bin
+
+for file in "$@"; do
+  tflint $file
+done