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

Update hashicorp/hcl and zclconfg/go-cty and hashicorp/go-getter to latest versions #677

Closed
Yantrio opened this issue Oct 6, 2023 · 1 comment · Fixed by #717
Closed
Assignees

Comments

@Yantrio
Copy link
Member

Yantrio commented Oct 6, 2023

Summary:

Upgrade the hashicorp/hcl and zclconf/go-cty packages for compatibility and enhanced functionality.
Details:

Actions

Justification:

  1. The upgrade is vital for alignment with recent changes that affect how marked and unmarked values are handled in both libraries. Specifically, the update pertains to improved error messages and functionality as noted in this GitHub Comment from the cty author. Upgrading to these versions will improve error reporting and potentially prevent certain types of bugs.
  2. The latest versions also contain the fixes that were used in Fix unknown conditional with sensitive branches erroring. #659
  3. The upgrade will change behaviour for the can and try, solving some possible bugs. See Allow can and try functions to return known results with unknown values in the arguments hashicorp/hcl#622 for more information
@Yantrio Yantrio changed the title Update hashicorp/hcl and zclconfg/go-cty to latest versions Update hashicorp/hcl and zclconfg/go-cty and go-releaser to latest versions Oct 11, 2023
@Yantrio Yantrio changed the title Update hashicorp/hcl and zclconfg/go-cty and go-releaser to latest versions Update hashicorp/hcl and zclconfg/go-cty and hashicorp/go-getter to latest versions Oct 11, 2023
@Yantrio
Copy link
Member Author

Yantrio commented Oct 11, 2023

We should also update hashicorp/go-getter

https://github.com/hashicorp/go-getter/releases/tag/v1.7.3

This should solve #676

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 a pull request may close this issue.

1 participant