-
Notifications
You must be signed in to change notification settings - Fork 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
[Python] Option to skip client validations #9719
Conversation
Thanks for this PR @slash-arun can you please fix failing tests, such as:
in order to merge them? |
@HugoMario Sorry for leaving this PR hanging, I was caught up with something else. I will fix these asap |
….com/slash-arun/swagger-codegen into py_option_to_skip_client_validations
@HugoMario @kenjones-cisco @frantuma This PR is ready for review. Please take a look. |
thanks a lot @slash-arun, this LGTM but would like also someone else review |
Sure @HugoMario , let me know if I should CC anyone else in particular. I already cc'd kenjones as per the document. |
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.
LGTM 👍
please build |
it seems the source repo was removed, i'm going to migrate changes to another PR in order to merge this |
PR checklist
./bin/
to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.sh
and./bin/security/{LANG}-petstore.sh
if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in.\bin\windows\
.3.0.0
branch for changes related to OpenAPI spec 3.0. Default:master
.Description of the PR