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

Add EasyAlign support for HCL files #182

Closed
wants to merge 1 commit into from
Closed

Add EasyAlign support for HCL files #182

wants to merge 1 commit into from

Conversation

jay7x
Copy link

@jay7x jay7x commented May 19, 2023

This allows to use EasyAlign in addition to Tabularize for HCL settings alignment

This commit allows to use EasyAlign in addition to Tabularize for HCL
settings alignment
@dimbleby
Copy link
Contributor

As you might have noticed, this repository is basically inactive - mostly because it's "done" rather than because it's "abandoned" - but still, I am reluctant to add features or expand existing ones: especially features that I don't use myself in code that I don't understand!

(And to be honest I never really understood the appeal of Tabularize or similar in a filetype that has an opinionated formatter. It's all going to look the same once you terraform fmt it, surely?)

So my immediate reaction is to lean towards "thanks, but no thanks". Would that be very annoying?

@jay7x
Copy link
Author

jay7x commented May 19, 2023

I'd be happy to use the terraform fmt but..

$ terraform fmt terragrunt.hcl 
╷
│ Error: Only .tf and .tfvars files can be processed with terraform fmt

If you know any HCL formatter please share, I'd go with it instead..

@jay7x
Copy link
Author

jay7x commented May 19, 2023

well.. after some experiments I found I can do terraform fmt - to format the terragrunt HCL.. maybe I'll create my own function to format on save instead then.. let's close the PR

@jay7x jay7x closed this May 19, 2023
@jay7x jay7x deleted the easyalign branch May 19, 2023 21:25
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.

None yet

2 participants