Skip to content
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.

Sign up for GitHub

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

Jump to bottom

gen: improve camel case to pascal #143

Closed
ernado opened this issue Feb 10, 2021 · 2 comments · Fixed by #175
Closed

gen: improve camel case to pascal #143

ernado opened this issue Feb 10, 2021 · 2 comments · Fixed by #175
Labels
area: gen Codegen issues good first issue Good for newcomers

Comments

@ernado
Copy link
Member

ernado commented Feb 10, 2021 •
edited

Upper version of cdnConfig should be CDNConfig, not CdnConfig.

In general, snake to pascal/camel is good, but lower camel to upper camel is bad.
Probably we should split words by upper case, like fooBarBaz to foo Bar Baz.
@ernado ernado added area: gen Codegen issues good first issue Good for newcomers labels Feb 10, 2021
@96RadhikaJadhav

This comment has been minimized.

Sign in to view
@ernado

This comment has been minimized.

Sign in to view
@ernado ernado mentioned this issue Mar 5, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: gen Codegen issues good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(gen): improve case handle ZorinArsenij/td
2 participants
@ernado @96RadhikaJadhav