-
Notifications
You must be signed in to change notification settings - Fork 22
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
Fix for args not using camel case in path #43
Conversation
@warmans there seem to be some lint errors |
oops, should be fixed now. |
still has some, run |
I don't think the CI integration is working properly. It's still complaining about stuff that is fixed (e.g. the substring callback arg no longer exists) 🤷♂️ |
I will look at it tomorrow :) |
src/parser.ts
Outdated
@@ -48,8 +48,11 @@ function parseMethods({paths, security, parameters}: Swagger): Method[] { | |||
), | |||
methodType: methodType.toUpperCase() as MethodType, | |||
parameters: transformParameters(operation.parameters, parameters || {}), | |||
// turn path interpolation `{this}` into string template `${args.this} | |||
// tslint:disable-next-line:no-invalid-template-strings |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this line can be removed, there is no more invalid template string warning
closes #42
This resolves the issue with incorrect case used in paths. There is an example of the issue in the github api.