Skip to content
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

Add require_data kwarg to validate() to make data optional #340

Merged
merged 2 commits into from Sep 26, 2019

Conversation

@cesarferradas
Copy link
Contributor

cesarferradas commented Sep 20, 2019

Proposed solution for issue #336.

@barneyjackson

This comment has been minimized.

Copy link

barneyjackson commented on 0cab804 Sep 20, 2019

Looks good to me!

flasgger/utils.py Outdated Show resolved Hide resolved
So that calls to validate() passing positional arguments
continue to work.
@cesarferradas cesarferradas force-pushed the allplants:master branch from 8d4ad5b to 22eb517 Sep 26, 2019
@billyrrr billyrrr merged commit a4e6fa4 into flasgger:master Sep 26, 2019
2 checks passed
2 checks passed
Travis CI - Pull Request Build Passed
Details
coverage/coveralls Coverage remained the same at 89.631%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.