-
Notifications
You must be signed in to change notification settings - Fork 2
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
Use Pydantic v1 API from v2 #27
Conversation
updates: - [github.com/psf/black: 23.3.0 → 23.7.0](psf/black@23.3.0...23.7.0) - [github.com/PyCQA/flake8: 6.0.0 → 6.1.0](PyCQA/flake8@6.0.0...6.1.0) - [github.com/asottile/pyupgrade: v3.4.0 → v3.10.1](asottile/pyupgrade@v3.4.0...v3.10.1)
c15c48e
to
273b83e
Compare
273b83e
to
7ab6620
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.
Looks good to me! I don't see anything wrong and if tests aren't complaining seems good.
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! I tried searching for other pydantic instances and didn't find anything. I also love the testing of both versions, great job!
Thanks both! Now to incrementally do the same for every other repo ... 😁 |
Description
This is an easier attempt at #26, enabling v2 to be installed while basically keeping the code the same, using
pydantic.v1
.Thanks for @jthorton for tipping us off to https://docs.pydantic.dev/latest/migration/#continue-using-pydantic-v1-features.
Todos
Notable points that this PR has either accomplished or will accomplish.
pydantic.v1
import pathQuestions
Status