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 category expansion out of restmapper #43622
Conversation
7aecee9
to
cfa1478
Compare
@k8s-bot verify test this |
cfa1478
to
8895f31
Compare
@fabianofranz I suspect you're pretty close to this at the moment given the discovery changes. I think I fixed the go vet problem. |
|
/lgtm |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: deads2k, fabianofranz
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
I don't think the overriding of "all" ever worked on our CLI since |
Automatic merge from submit-queue (batch tested with PRs 42087, 43383, 43622) |
RESTMapping isn't related to CategoryExpansion (the bit that expands "all" into items to be RESTMapped). This provides that separation and simplifies the RESTMapper interface.
@kubernetes/sig-cli-pr-reviews