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

Output directory tflint is executed from #161

Closed
varac opened this issue Dec 2, 2020 · 1 comment · Fixed by #234
Closed

Output directory tflint is executed from #161

varac opened this issue Dec 2, 2020 · 1 comment · Fixed by #234
Assignees
Labels
estimate/1h Need 1 hour to be done feature New feature or request hook/terraform_tflint Bash hook

Comments

@varac
Copy link

varac commented Dec 2, 2020

See also terraform-linters/tflint#991

When using the tflint hook I get an output like this:

Warning: Module should include an empty outputs.tf file (terraform_standard_module_structure)

  on outputs.tf line 1:
   (source code not available)

Reference: https://github.com/terraform-linters/tflint/blob/v0.21.0/docs/rules/terraform_standard_module_structure.md

1 issue(s) found:

Warning: Module should include an empty outputs.tf file (terraform_standard_module_structure)

  on outputs.tf line 1:
   (source code not available)

Reference: https://github.com/terraform-linters/tflint/blob/v0.21.0/docs/rules/terraform_standard_module_structure.md

I'm unsure if tflint will include the path because this is a particular issue how the pre-commit hooks uses tflint, so I suggest to fix it here. Please output the directory tflint is executed from.

varac added a commit to varac/pre-commit-terraform that referenced this issue Dec 2, 2020
@varac
Copy link
Author

varac commented Dec 2, 2020

I made #162 for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
estimate/1h Need 1 hour to be done feature New feature or request hook/terraform_tflint Bash hook
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants