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

chore(deps): update dependencies #3797

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

chore(deps): update dependencies #3797

wants to merge 3 commits into from

Conversation

uluzox
Copy link

@uluzox uluzox commented Jan 3, 2025

in particular remove tmccombs/hcl2json replacement cdktf/hcl2json and update the library as the issues are now fixed

Solves: #3793

Related issue

Fixes #3793

Description

Dependency https://github.com/tmccombs/hcl2json was replaced with github.com/cdktf/hcl2json
https://github.com/hashicorp/terraform-cdk/blob/c1ca5b3896b2751a7297da97a1bd9eb10d2ffddf/packages/%40cdktf/hcl2json/go.mod#L11C48-L11C73

in order to correctly keep escape sequences for interpolations in template expressions
tmccombs/hcl2json@cc85d3b

This was now fixed in tmccombs/hcl2json#96 thus the original library could be used.

Also v0.6.5 of the library probably solves parsing provider functions tmccombs/hcl2json#110

Checklist

  • I have updated the PR title to match CDKTF's style guide
  • I have run the linter on my code locally
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation if applicable
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works if applicable
  • New and existing unit tests pass locally with my changes

in particular remove tmccombs/hcl2json replacement cdktf/hcl2json
and update the library as the issues are now fixed

Solves: hashicorp#3793
Copy link

hashicorp-cla-app bot commented Jan 3, 2025

CLA assistant check
All committers have signed the CLA.

@uluzox uluzox marked this pull request as ready for review January 3, 2025 19:06
@uluzox uluzox requested a review from a team as a code owner January 3, 2025 19:06
@uluzox uluzox requested review from mutahhir and DanielMSchmidt and removed request for a team January 3, 2025 19:06
Copy link
Contributor

@DanielMSchmidt DanielMSchmidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds like a good idea!

go.work Outdated Show resolved Hide resolved
packages/@cdktf/hcl2json/go.mod Outdated Show resolved Hide resolved
@DanielMSchmidt
Copy link
Contributor

I think this currently prevents the build to succeed

@uluzox
Copy link
Author

uluzox commented Jan 6, 2025

I think this currently prevents the build to succeed

I updated the PR and also set toolchain to Go Version without the patch part of the semantic version.

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.

COMPONENT: use latest version of tmccombs/hcl2json
2 participants