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

fix: Fixed terraform_fmt with tfenv, when terraform default version is not specified #389

Merged
merged 3 commits into from
May 25, 2022

Conversation

MaxymVlasov
Copy link
Collaborator

@MaxymVlasov MaxymVlasov commented May 25, 2022

Put an x into the box if that apply:

  • This PR introduces breaking change.
  • This PR fixes a bug.
  • This PR adds new functionality.
  • This PR enhances existing functionality.

Description of your changes

Fixes #379

How can we test changes

repos:
- repo: https://github.com/antonbabenko/pre-commit-terraform
  rev: d0c781e98a310b1b03900280feaf630832bef332
  hooks:
    - id: terraform_fmt

Also, test details exist in #379 (comment), and the same tests was performed for this PR too.

@MaxymVlasov MaxymVlasov requested a review from antonbabenko May 25, 2022 13:06
@MaxymVlasov MaxymVlasov requested a review from yermulnik as a code owner May 25, 2022 13:06
@MaxymVlasov MaxymVlasov self-assigned this May 25, 2022
@antonbabenko antonbabenko changed the title fix: Able work terraform_fmt with tfenv, when terraform default version not specified fix: Fixed terraform_fmt with tfenv, when terraform default version is not specified May 25, 2022
@antonbabenko antonbabenko merged commit 1b9476a into master May 25, 2022
@antonbabenko antonbabenko deleted the chore/refactor_terraform_fmt branch May 25, 2022 13:38
antonbabenko pushed a commit that referenced this pull request May 25, 2022
## [1.72.1](v1.72.0...v1.72.1) (2022-05-25)

### Bug Fixes

* Fixed `terraform_fmt` with `tfenv`, when `terraform` default version is not specified ([#389](#389)) ([1b9476a](1b9476a))
@antonbabenko
Copy link
Owner

This PR is included in version 1.72.1 🎉

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

Successfully merging this pull request may close these issues.

[tfenv without default version] terraform_fmt failed when directory contains only tfvars
3 participants