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

enhancement: TFLint - fix deprecated argument in ".tflint.hcl" and other improvements #3688

Merged
merged 7 commits into from
Jun 24, 2024

Conversation

TommyE123
Copy link
Contributor

TFLint fix deprecated argument in ".tflint.hcl"

Fixes #3618

Context:

@mg-alanjones kindly highlighted a warning appearing when running TFLINT.
WARNING: "module" attribute in /action/lib/.automation/.tflint.hcl is deprecated. Use "call_module_type" instead.
I’ve fixed this along with some improvements.

Changes:

In the megalinter default template:
• Updated the commented url https://github.com/terraform-linters/tflint/blob/master/docs/user-guide/config.md.
• Switched from module = false to call_module_type = "local".
• Added more plugins so it now supports AWS, AZUREARM and GOOGLE.
Other changes:
• Added more tests to support the 3 languages.
• Documentation to suggest using a config refined to a language.

Testing:

AZURE PIPELINE – Terraform project using megalinter default template with forced linting error.

Before:
image
image

After:
image

Note appears in tflint page

image

If there are any issues or concerns, please let me know, and I will address them promptly.
Thank you for reviewing this PR and I look forward to your feedback.
Tom

@echoix
Copy link
Collaborator

echoix commented Jun 24, 2024

Looks great!

@echoix echoix merged commit 7ae1cfe into oxsecurity:main Jun 24, 2024
6 checks passed
@nvuillam
Copy link
Member

.... and @TommyE123 strikes again ! :) Thanks :)

@TommyE123 TommyE123 deleted the TFLint-deprecated-argument branch July 4, 2024 20:18
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.

TFLint - deprecated argument in ".tflint.hcl"
3 participants