Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Replace hcl2 with hcl/v2 #112

Closed
wants to merge 1 commit into from
Closed

Replace hcl2 with hcl/v2 #112

wants to merge 1 commit into from

Conversation

myyra
Copy link
Contributor

@myyra myyra commented Jul 5, 2023

The hcl2 repo was only used for the development phase of HCL 2 and is now archived.

This PR replaces the use of github.com/hashicorp/hcl2 with github.com/hashicorp/hcl/v2. The APIs are compatible, and the only change is better output formatting (at least subjectively) when using --tf12format, where it now adds a newline after every opening bracket.

Note: some of the files had unrelated changes when using go fmt, which I chose not to include in this PR. Please let me know if I should add the formatted changes as well.

The github.com/hashicorp/hcl2 package was used temporarily for the
development of HCL 2, and is now archived. This commit replaces with the
API-compatible github.com/hashicorp/hcl/v2.

As a result of this, the output of --tf12format will change slightly,
with newlines added after every open bracket.
@sl1pm4t
Copy link
Owner

sl1pm4t commented Jul 5, 2023

Thanks for this. For some reason the drone build system is hanging, so tests and builds are not running.
Looks like it's finally time to update this repo to use GH Actions before I'll be able to accept this change.

@sl1pm4t
Copy link
Owner

sl1pm4t commented May 18, 2024

Thanks @myyra - I finally merged this after some changes in #119

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

Successfully merging this pull request may close these issues.

2 participants