-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
code-generator: complete PkgName, GroupName, GoName seperation #55614
code-generator: complete PkgName, GroupName, GoName seperation #55614
Conversation
- GroupNames are domain-like logical name for the group. Only the first segment is used as default for GoName - PkgName is the directory name. All packages in client, informer, lister re-use this for packages. - GoName is the Go identifier (CamelCase) used to reference the group, e.g. in the interface names, in the clientsets etc. Moreover it is used for package import aliases. The GoName defaults to the first segment of the GroupName.
db36898
to
2779de5
Compare
Generated code looks good. /lgtm |
/approve no-issue |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mfojtik, sttts Associated issue requirement bypassed by: sttts The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Automatic merge from submit-queue (batch tested with PRs 54436, 53148, 55153, 55614, 55484). If you want to cherry-pick this change to another branch, please follow the instructions here. |
... in client-gen, informer-gen, lister-gen. Follow-up of #54950.
Before this PR, the generated code was broken for internal types and for group package names that were no valid Go identifiers.
This PR completes the separation in the following sense:
Note: this PR does not change the generated code in Kubernetes, only the examples in k8s.io/code-generator.