Skip to content

ci: add tflint#95

Merged
babinskiy merged 4 commits intomasterfrom
ci/tf-lint
Mar 21, 2025
Merged

ci: add tflint#95
babinskiy merged 4 commits intomasterfrom
ci/tf-lint

Conversation

@babinskiy
Copy link
Contributor

@babinskiy babinskiy commented Mar 21, 2025

This pull request introduces a new Terraform linting job to the CI workflow and adds a configuration file for TFLint. The most important changes include adding a new tflint job in the CI workflow and configuring TFLint with specific plugins and settings.

CI Workflow Enhancements:

  • .github/workflows/ci.yml: Added a new tflint job that runs on ubuntu-latest, sets up TFLint, and runs it with specific options.

TFLint Configuration:

  • .tflint.hcl: Added a new configuration file for TFLint specifying required version, plugin directory, and enabling plugins for Terraform, AWS, and AzureRM with specific versions and sources.

@babinskiy babinskiy requested a review from Veetaha March 21, 2025 16:28
@babinskiy babinskiy self-assigned this Mar 21, 2025
@babinskiy babinskiy marked this pull request as ready for review March 21, 2025 16:30
@babinskiy babinskiy force-pushed the ci/tf-lint branch 2 times, most recently from 0873443 to 00acd22 Compare March 21, 2025 17:41
runs-on: ubuntu-latest

env:
TFLINT_CONFIG_FILE: "${{ github.workspace }}/.tflint.hcl"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If TFLint runs in less than a second, consider adding it to the .githooks/pre-commit

update git hook
@babinskiy babinskiy merged commit 0fe9a56 into master Mar 21, 2025
7 checks passed
@babinskiy babinskiy deleted the ci/tf-lint branch March 21, 2025 18:14
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.

3 participants