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

Upgrade to pydantic v2 #2

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

oliviagyg
Copy link

@oliviagyg oliviagyg commented Nov 14, 2023

This library unfortunately stopped working after pydantic upgraded to V2. I've made the necessary changes here to provide support for pydantic v2. This means pydantic v2 would be required for future versions (i.e. pydantic v1 wouldn't work), which I don't think is necessarily a problem, but it's definitely something to keep in mind.

If you could take a look over the changes and merge if they look good to you, I'd appreciate it -- I'm upgrading a project to use pydantic v2 and would appreciate still being able to use this library :)

@oliviagyg
Copy link
Author

@albertnadal Could you please take a look at this?

@dashedman
Copy link

vote on
@albertnadal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants