-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 support for checkov for linting terraform files #4006
Conversation
When you add new linters or fixers, make sure to add them into the tables in supported-tools.md and ale-supported-languages-and-tools.txt. These files must be sorted properly so ensure the tool is added in the proper alphabetical order. |
Also add an entry to ale-terraform.txt with the new linter and configuration options. |
@hsanson Thank you for the feedback and the explanations! I updated the branch accordingly and hope, it is now ready for being merged. |
@thyme-87 sorry, forgot to mention you have to add checkov entry to the list of tools in ale.txt file. Make sure to add it in the same order as in the ale-terraform.txt file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution.
Display rule violations from checkov via
ale
. Motivated by checkov2vim which I did not manage to set up correctly on my machine.I provided the most basic tests (derived from the tests for the
terraform
linter andansible_lint
) and I do hope that this is sufficient.Please let me know if I missed anything.