-
Notifications
You must be signed in to change notification settings - Fork 15
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
avrogo generates non-idiomatic names #126
Comments
Hello, and thanks for the suggestion. This is a reasonable request so please feel free to submit a PR |
Hello, please could someone review the PR I opened? |
Hello @skateinmars. Do you think you would be able to review this PR please ? |
Hi |
Names produced by
avrogo
do not follow idiomatic Go capitalisation/initialisation rules. (e.g. useID
rather thanid
orId
).This can be reproduced with the following schema
avrogo
generatesbut an idiomatic representation would be
There are third-party libraries available to achieve this (e.g. github.com/hamba/avro uses github.com/ettle/strcase's PascalCase).
If this were implemented then it would be helpful to allow additional initialisms with command line arguments. For example in my use case I have schemas referring to
MW
(megawatts) and would like that initialised too.I'm happy to work on this but I want to check if the change would be accepted before starting.
The text was updated successfully, but these errors were encountered: