-
-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Issue in swagger migration guide #150
Comments
@delenius thanks for the feedback. That section aims to keep our users up-to-date regarding some mustache variable renaming. If you're not using customized templates, you do not need to worry about that. I'll add a line to explain that's for using customized templates only. |
Yes but it says "replace X with X".
…On Fri, May 25, 2018 at 8:06 AM, William Cheng ***@***.***> wrote:
@delenius <https://github.com/delenius> thanks for the feedback.
That section aims to keep our users up-to-date regarding some mustache
variable renaming. If you're not using customized templates, you do not
need to worry about that.
I'll add a line to explain that's for using customized templates only.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#150 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AHkajaO_btCdgyi1umqhxrYRNeA3XaVhks5t2B3cgaJpZM4UOHGF>
.
|
Oh, I see the lower vs uppercase "t" now, but only in the first case. It still says |
Good catch. I'll fix that. |
UPDATE: I've filed #152 to fix it. |
Closed via #152. Thanks again for the feedback. |
In the swagger-codegen migration guide, there is this confusing section. I'm actually not sure what it's supposed to say, or I would submit a PR:
The text was updated successfully, but these errors were encountered: