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
move ObjectEnum from console-shared to kubevirt #9444
move ObjectEnum from console-shared to kubevirt #9444
Conversation
@spadgett @vojtechszocs @christianvogt please review |
71078e5
to
79d09b8
Compare
79d09b8
to
509e9fb
Compare
@glekner I don't think this is a valid API for the console plugin SDK to expose. It is a utility to help with enums but it doesn't help the plugin developer to build plugins. I suggest copying the code to your repo. We want to expose API that are required to build plugins, help them work with k8s APIs, and build a consistent user interface. |
@christianvogt yep, moved the code into kubevirt |
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
/hold for others to look as well
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: glekner, spadgett 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 |
/hold cancel i'm fine with the move |
No description provided.