-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
⚠️ officially deprecate go/v2 plugin (legacy layout Kubebuilder 2.x) #2734
⚠️ officially deprecate go/v2 plugin (legacy layout Kubebuilder 2.x) #2734
Conversation
Skipping CI for Draft Pull Request. |
/hold To get the reviews only |
6ae4507
to
580b841
Compare
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.
Just some minor wording changes
/tide merge-method-squash |
c06f4a1
to
18e2441
Compare
/hold cancel |
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.
Should we add deprecation warning in documentation as well?
Here are the links:
HI @rashmigottipati,
The info is in the doc where the other plugins are doc already so I do not think that is required here. OR Is it about the warning when users run the commands and are using the old legacy plugin? |
I agree here with @rashmigottipati that it would be a good idea to update all the documentation that mentions the I do see that in the documentation it already recommends not using it unless you need the legacy layout, but I think adding a deprecated marking would still be good since it gives warning in the documentation the the I think just simply adding something like |
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.
I'm good with adding [Deprecated]
next to all references in the docs based on @everettraven's suggestion.
It will be nice to add this warning before completely removing support for the go/v2
plugin.
18e2441
to
44ee14b
Compare
44ee14b
to
3021b09
Compare
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.
/lgtm
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.
Some minor wording nits
/hold |
Co-authored-by: Bryce Palmer <everettraven@gmail.com>
12d40e8
to
e83c9d7
Compare
Co-authored-by: Bryce Palmer <everettraven@gmail.com>
/label tide/merge-method-squash |
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.
/lgtm
@everettraven: changing LGTM is restricted to collaborators In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
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.
/hold cancel
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: camilamacedo86, everettraven, rashmigottipati 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 |
Description
officially deprecate go/v2 plugin
Motivation
Kubebuilder 2.x
layoutCloses: #2705