-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Apiextensions DSL uses NonNamespaceOperation for CustomResourceDefinitions #2747
Apiextensions DSL uses NonNamespaceOperation for CustomResourceDefinitions #2747
Conversation
ohk, let's hold merge. |
Hi, I think that since this PR just fixes a bug, we can safely merge it. It makes sense to make the DSL consistent with the API even if this is going to be refactored in the future. Refactoring should be covered in future PRs. |
…tions CustomResourceDefinitions api DSL should not have inNamespace exposed. This was due to my mistake while adding apiextensions() DSL in fabric8io#2359
12b5daa
to
23319fc
Compare
I agree with Marc here we can safely merge this PR. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
CustomResourceDefinitions api DSL should not have inNamespace exposed.
This was due to my mistake while adding apiextensions() DSL in
#2359
Description
Type of change
test, version modification, documentation, etc.)
Checklist