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

Issue 83: Support terraform fmt #298

Merged
merged 27 commits into from
Sep 14, 2020
Merged

Issue 83: Support terraform fmt #298

merged 27 commits into from
Sep 14, 2020

Conversation

kmanning
Copy link
Collaborator

  • Issue Support terraform fmt #83 - add terraform fmt as part of the TerraformValidateStage.
  • Support optional recursive directory check
  • Support displaying diffs
  • Ensure support for both terraform 0.11.x and 0.12.x

@codecov-commenter
Copy link

codecov-commenter commented Sep 12, 2020

Codecov Report

Merging #298 into master will increase coverage by 1.89%.
The diff coverage is 86.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #298      +/-   ##
==========================================
+ Coverage   76.54%   78.43%   +1.89%     
==========================================
  Files          39       41       +2     
  Lines        1006     1048      +42     
  Branches      235      244       +9     
==========================================
+ Hits          770      822      +52     
+ Misses        122      105      -17     
- Partials      114      121       +7     
Impacted Files Coverage Δ
src/TerraformValidateStage.groovy 81.81% <ø> (+54.54%) ⬆️
src/ValidateFormatPlugin.groovy 71.42% <71.42%> (ø)
src/TerraformFormatCommand.groovy 86.11% <86.11%> (ø)
src/TerraformPlugin.groovy 95.00% <100.00%> (+4.09%) ⬆️
src/TerraformPluginVersion12.groovy 86.66% <100.00%> (+28.33%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4aa1f98...8db33be. Read the comment docs.

@kmanning kmanning added this to the v5.11 milestone Sep 14, 2020
@kmanning kmanning linked an issue Sep 14, 2020 that may be closed by this pull request
@kmanning kmanning merged commit 3540d0b into manheim:master Sep 14, 2020
@kmanning kmanning deleted the issue_83 branch April 20, 2021 13:57
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.

Support terraform fmt
2 participants