-
Notifications
You must be signed in to change notification settings - Fork 32
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
feat(dynamite_petstore_example): Init #1137
Conversation
@Leptopoda I can move the fix for the security schemes to a separate PR. With the applied patch it is not necessary anyway since it doesn't break this PR. |
e66e38a
to
dc6705f
Compare
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.
Please also open followup issues for:
- the authentication issue
- to test the petstore against the server
yq is failing because of different implementations... I'll find a solution for that |
dc6705f
to
f242f30
Compare
I'm now using the petstore specification from https://github.com/OAI/OpenAPI-Specification which doesn't have the authentication problems and doesn't need any patching or yq magic. |
f242f30
to
6de2061
Compare
Be aware that this uses the utf8 codec so #1143 is conflicting. |
Signed-off-by: jld3103 <[email protected]>
6de2061
to
3e5e53f
Compare
Depends on #1136
#761