Update the module name #1390

joaopapereira commented 5 months ago

What this PR does / why we need it: updates the module name to carvel.dev/kapp-controller and updates vendir to match the new module name

Which issue(s) this PR fixes:

Fixes #1378

Additional Notes for your reviewer:

Review Checklist:

[ ] Follows the developer guidelines
[ ] Relevant tests are added or updated
[ ] Relevant docs in this repo added or updated
[ ] Relevant carvel.dev docs added or updated in a separate PR and there's a link to that PR
[ ] Code is at least as readable and maintainable as it was before this change

Additional documentation e.g., Proposal, usage docs, etc.:

joaopapereira commented 4 months ago

When we release 0.52.0 we should merge the PR and create version 0.53.0

praveenrewar commented 4 months ago

I was thinking that we should make a release now with this change. We currently have 3 commits after the last release, 2 of them are bumps and one of them is a fix for using kctrl as a library, so ideally one would want to bump to this release for those changes. We could also wait, but then there's a chance that there will be merge conflicts on this PR and we would have to again spend some time on resolving them.

joaopapereira commented 4 months ago

I am ok with releasing it. On Monday if y'll have some time can you release it and merge this PR?

praveenrewar commented 4 months ago

Sure, I will do it.

carvel-dev / kapp-controller