Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a terraform linter #2529

Merged
merged 1 commit into from
May 29, 2019
Merged

Add a terraform linter #2529

merged 1 commit into from
May 29, 2019

Conversation

maxwell-k
Copy link
Contributor

This linter uses the check functionality built into terraform. ALE
already has a fixer using terraform fmt but this doesn't provide error
messages. ALE already has a linter using tflint but this requires an
extra application to be installed.

For example this linter will give a warning that ! is an illegal
character in the line below:

variable "example" !{}

This linter runs the buffer through the command below and parses the
output:

terraform fmt -no-color -check=true -

This commit includes a basic implementation, documentation and tests.
The only option is to control which executable is run.

Tested with:

$ terraform -version
Terraform v0.11.13

This linter uses the check functionality built into terraform. ALE
already has a fixer using `terraform fmt` but this doesn't provide error
messages. ALE already has a linter using `tflint` but this requires an
extra application to be installed.

For example this linter will give a warning that ! is an illegal
character in the line below:

    variable "example" !{}

This linter runs the buffer through the command below and parses the
output:

    terraform fmt -no-color -check=true -

This commit includes a basic implementation, documentation and tests.
The only option is to control which executable is run.

Tested with:

    $ terraform -version
    Terraform v0.11.13
@w0rp w0rp merged commit 4f02edc into dense-analysis:master May 29, 2019
@w0rp
Copy link
Member

w0rp commented May 29, 2019

Cheers! 🍻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants