-
Notifications
You must be signed in to change notification settings - Fork 91
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
⚠️ Remove v1alpha2 types #164
Conversation
/test generate |
f463602
to
daaf599
Compare
/cc @maelk |
/hold |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fmuyassarov, maelk 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 |
/test-integration |
lgtm, waiting for the CI to pass |
/test-integration |
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.
Tests have passed now, so merging it.
/lgtm
/test-centos-integration |
/test-centos-integration |
PLEASE DON'T MERGE THIS. We are going to have lazy consensus of two weeks to check if there are any objections on this.
|
daaf599
to
f57a716
Compare
f57a716
to
8ccec64
Compare
/test golint |
/test generate |
Remove v1alpha2 types
8ccec64
to
b2cbd5f
Compare
/test golint |
/test-integration |
Since the agreed lazy consensus date reached and there were no objections AFAIK, this should be safe to merge. @fmuyassarov leaving it up to you to unhold. /lgtm |
/hold cancel |
/test golint |
/test unit |
What this PR does / why we need it:
Remove v1alpha2 types.
Fixes: #159