-
Notifications
You must be signed in to change notification settings - Fork 71
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 support for pydantic v2 #144
Conversation
After a small fight with the CI config and the package version, I think this is ready to go. Some more package could be updated, but since this is in a working state, it's probably better to keep it for another PR |
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.
This is awesome! Thank you so much! 💯
I've created a release, should be live soon. Thanks for all your PRs and help here once again, @enadeau 😊 |
* update dependencies * update trio to fix a test issue * fix pydantic v2 compatibility issue * add pydantic v2 to ci * setup pydantic version before fastapi version in ci * remove pydantic settings before downgrading to pydantic v1 * update python patch version in ci * update pytest-aiohttp * no cover for line only run by pydantic v1
I've made the smallest changes I could to get the project running under Pydantic v1 and v2. I've also added the the two version of pydantic the testing matrix.