-
Notifications
You must be signed in to change notification settings - Fork 14
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
project: kubebuilder layout v3 -> v4 #1160
Conversation
expected to fail in multiple lanes |
118b628
to
120a3b5
Compare
should not be horribly broken anymore |
43aae27
to
d206124
Compare
/retest-required |
d2255a1
to
0ffa6aa
Compare
We have no good reason to get stuck on layout v3 and make our time harder when consuming upcoming changes and fixes. Most of the changes here stem from the fact we change the path of the API package. We also consume as much as possible from the new Makefile features. done using https://book.kubebuilder.io/migration/migration_guide_gov3_to_gov4 Signed-off-by: Francesco Romani <[email protected]>
0ffa6aa
to
643c760
Compare
/cc @Tal-or |
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.
Thanks
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ffromani, Tal-or The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
We have no good reason to get stuck on layout v3 and make our time harder when consuming upcoming changes and fixes.
Most of the changes here stem from the fact we change the path of the API package.
We also consume as much as possible from the new Makefile features.
Done using https://book.kubebuilder.io/migration/migration_guide_gov3_to_gov4