-
Notifications
You must be signed in to change notification settings - Fork 40.1k
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
PluginFactoryArgs: Expose explicit fields #20254
Conversation
None of the embedded fields are used for inheritance methods. Expose them for better code analysis.
Labelling this PR as size/S |
@davidopp |
LGTM cc/ @davidopp for last check? |
@k8s-bot test this issue: #IGNORE Tests have been pending for 24 hours |
GCE e2e test build/test passed for commit 70408d2. |
The author of this PR is not in the whitelist for merge, can one of the admins add the 'ok-to-merge' label? |
@k8s-bot test this issue: #IGNORE Tests have been pending for 24 hours |
1 similar comment
@k8s-bot test this issue: #IGNORE Tests have been pending for 24 hours |
GCE e2e test build/test passed for commit 70408d2. |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e test build/test passed for commit 70408d2. |
Automatic merge from submit-queue |
Auto commit by PR queue bot
None of the embedded fields are used for inheritance methods. Expose
them for better code analysis.