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
Update setup.py and version #51
Conversation
dwinston
commented
Jun 21, 2019
•
edited
edited
- Bumps internal minor version (0.1.x to 0.2.0) for release to PyPi
- Updates setup.py metadata to reflect current authorship
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.
thanks @dwinston !
Oh, I see, there's new validation errors... |
Interesting. The schema generates without error for fastapi 0.29.1, but not for fastapi 0.20.0. I'll see what's going on... |
Okay, I think it's this reported bug in pydantic 0.28. I implemented the two-line proposed fix to pydantic, and the schema generates properly in this case. So, we need to wait for the fix to pydantic (and corresponding updated dep in fastapi) to upgrade fastapi. Whew! |
@dwinston did you still want to push? |
I’d like to wait for the fastapi/pydantic fix upstream and then update this PR with the later version(s) so that the tests pass. |
Forget it -- I've removed the non-fastapi updates so that this passes tests and we can get updated setup.py info and push a new version to PyPI. |
lgtm, merge whenever you're ready |