git-hooks.nix icon indicating copy to clipboard operation
git-hooks.nix copied to clipboard

Remove `terraform-fmt` wrapper

Open lbodor opened this issue 1 year ago • 3 comments

Since the time terraform-fmt wrapper was added to git-hooks.nix 5 years ago, both terraform and tofu now support formatting multiple files.

https://github.com/hashicorp/terraform/pull/31286 https://github.com/hashicorp/terraform/pull/28191

lbodor avatar Jul 15 '24 01:07 lbodor

I just noticed https://github.com/cachix/git-hooks.nix/pull/419, which also makes the change to format individual files, rather than entire directories, like the existing terraform-fmt wrapper.

lbodor avatar Jul 15 '24 02:07 lbodor

Should we rather use #419?

domenkozar avatar Jul 15 '24 10:07 domenkozar

Should we rather use #419?

Sure, that would be fine.

lbodor avatar Jul 15 '24 19:07 lbodor

Merged #419, which is practically identical to this.

sandydoo avatar Jan 21 '25 13:01 sandydoo