-
-
Notifications
You must be signed in to change notification settings - Fork 6.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
[go] respect initialisms or acronyms in names #4900
Conversation
👍 Thanks for opening this issue! The team will review the labels and make any necessary changes. |
Thanks for the PR. Please PM me via Slack when you've time. |
@wing328 Sorry, I don't have Slack. Maybe mail/Hangout? tomasz[dot]rup[at]gmail[dot]com |
Sure let's do it via Google Hangout tomorrow (Friday). Please ping me (wing328hk@gmail.com) when you're free. My timezone is +0800. |
Is Does this functionality, changing client APIs, need to be behind a generator user flag? |
Any updates? |
we've added name mapping support to the go generators which should allow users to address issues mentioned in this PR: https://github.com/OpenAPITools/openapi-generator/blob/master/docs/customization.md#name-mapping please give it a try with the latest master. |
Resolved #4833
PR checklist
./bin/
(or Windows batch scripts under.\bin\windows
) to update Petstore samples related to your fix. This is important, as CI jobs will verify all generator outputs of your HEAD commit, and these must match the expectations made by your contribution. You only need to run./bin/{LANG}-petstore.sh
,./bin/openapi3/{LANG}-petstore.sh
if updating the code or mustache templates for a language ({LANG}
) (e.g. php, ruby, python, etc).master
,4.3.x
,5.0.x
. Default:master
.