-
-
Notifications
You must be signed in to change notification settings - Fork 998
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
CLI Redesign #3445
Comments
It's great to see this RFC, this would surely clean up some mess and make Terragrunt more clear to new users. That said, it's not super clear in the text how the new For my two cents on specific changes:
|
Thanks for the feedback @slaughtr !
|
It's a small comment, but if we decide it's worth continuing to namespace our flags, maybe using |
One other idea: If users find the difference between For example, We could even allow running |
I've attempted to address both your concerns with these two updates to the RFC, let me know if you have any more feedback! Personally, I don't think Terragrunt should be namespacing its own flags at all. No other tool I know of does this, and it makes the experience of using Terragrunt worse. The proposal provides tools for disambiguating flags like |
Y'know, with those extra examples and your points above I now agree. |
+1 to @slaughtr's comment! |
I like these changes. Terragrunt flags have been difficult to remember, for me, because of the namespacing. One thing that has been on my wishlist for |
An update on this RFC: Due to the confusion in issue #3533, the RFC has been updated to change the flag name It seems like the community is largely OK with the changes in this RFC otherwise. For transparency, I'm intentionally leaving this RFC in If you have any major objections to this RFC make them known soon! |
With the conclusion of #3134 , this RFC is now ready to be marked accepted, and for work to be scheduled. We'll look to start releasing progress on in it in the near future (within the next two weeks). |
Summary
This RFC was initially vetted in a Gruntwork internal Notion page, and increased in size and complexity until it became detailed enough that it was no longer practical to publish as a GitHub issue.
To read the RFC, navigate to the publicly published version of the Notion page here:
https://gruntwork.notion.site/PUBLIC-Terragrunt-CLI-Redesign-10ed8759a6828042ac6bc03aef4d9dfb
That page will be updated according to feedback, with comments placed on this issue to track those updates.
All feedback, conversation and updates will take place on this issue.
The text was updated successfully, but these errors were encountered: