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

Proxies handle chdir flag #247

Merged
merged 4 commits into from
Sep 8, 2024
Merged

Proxies handle chdir flag #247

merged 4 commits into from
Sep 8, 2024

Conversation

dvaumoron
Copy link
Contributor

Resolve #246

Signed-off-by: Denis Vaumoron <dvaumoron@gmail.com>
Signed-off-by: Denis Vaumoron <dvaumoron@gmail.com>
Signed-off-by: Denis Vaumoron <dvaumoron@gmail.com>
Signed-off-by: Denis Vaumoron <dvaumoron@gmail.com>
@dvaumoron dvaumoron merged commit e37ffe6 into main Sep 8, 2024
4 checks passed
@OJFord
Copy link

OJFord commented Sep 9, 2024

🚀 thanks @dvaumoron!

@rdhar
Copy link
Contributor

rdhar commented Sep 9, 2024

Thanks for this update, @dvaumoron. I wanted to clarify whether the required_version is now a requirement to have?

Asking because I have some simply-but-valid Terraform configs (for example) without defining any providers/versions which now fail as per this PR from tenv v3.2.0.

It is by no means a major problem: setting the TFENV_TERRAFORM_VERSION addresses this fine. Though, I wonder if it's worth highlighting this in the changelog.

Edit: More than happy to confirm this is resolved as of v3.2.2, thank you!

@dvaumoron dvaumoron deleted the feat/proxy-with-chdir-flag branch October 25, 2024 15:11
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.

Does not work with terraform -chdir or modules
3 participants