Skip to content

feat: Remove deprecated variables#100

Draft
gabops wants to merge 3 commits intogruntwork-io:mainfrom
gabops:feat-remove-deprecated-variables
Draft

feat: Remove deprecated variables#100
gabops wants to merge 3 commits intogruntwork-io:mainfrom
gabops:feat-remove-deprecated-variables

Conversation

@gabops
Copy link

@gabops gabops commented Apr 28, 2025

Description

Remove deprecated variables

BREAKING CHANGE: Terragrunt on any consumer using version <= v0.48.6 will cease to behave as expected.

Release Notes (draft)

  • Removed TF_INPUT in favor of TG_NON_INTERACTIVE

  • Removed TERRAGRUNT_TFPATH in favor of TG_TF_PATH

Migration Guide

Update to Terragrunt >= v0.48.6

BREAKING CHANGE: Terragrunt on any consumer using version <= v0.48.6
will cease to behave as expected.
@yhakbar
Copy link
Contributor

yhakbar commented May 9, 2025

@gabops this forces a breaking change to avoid some warnings, which might not be in the best interest of the community.

Have you considered an alternate approach where the version of Terragrunt is checked, and the environment variable set is adjusted accordingly? That would provide a backwards compatible change here.

@gabops
Copy link
Author

gabops commented May 12, 2025

@gabops this forces a breaking change to avoid some warnings, which might not be in the best interest of the community.

Have you considered an alternate approach where the version of Terragrunt is checked, and the environment variable set is adjusted accordingly? That would provide a backwards compatible change here.

Yes, I considered that, however I decided to go ahead with the breaking change assuming the resulting version with my change to be 3.0.0 which would be fine from the consumer's point of view. Anyways, I'll add the logic for conditionally supporting both variables based on the version. I'll push something at some point of today.

@gabops
Copy link
Author

gabops commented May 21, 2025

@gabops this forces a breaking change to avoid some warnings, which might not be in the best interest of the community.

Have you considered an alternate approach where the version of Terragrunt is checked, and the environment variable set is adjusted accordingly? That would provide a backwards compatible change here.

Hi @yhakbar,

Sorry for the delay on this. As asked, I've made a change that keeps backwards compatibility. Not the most optimal way, however, it should do the job. Let me know what you think.

Thank you in advance.

Regards

@yhakbar
Copy link
Contributor

yhakbar commented Jul 29, 2025

Sorry I didn't get to this earlier, @gabops , do you mind rebasing this PR then re-requesting review?

@yhakbar yhakbar marked this pull request as draft July 29, 2025 16:35
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.

3 participants