-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
refactor: Remove depnameShort of terra* and kustomize #8388
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is possibly a breaking, becuase user configs can use depNameShort
which we don't know.
There is also no easy migration.
@rarkins How should we handle the backward compability? Try to do some config migration? Migration in template processing? |
If want to do it in steps then we can do this:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be merge now as we already have #8401 merged
🎉 This PR is included in version 24.39.2 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes:
Remove depNameShort of:
Context:
#8075
Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via: