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

chore(python): Pin pydantic in dev requirements <2.4.0 #11312

Merged
merged 1 commit into from
Sep 25, 2023
Merged

Conversation

stinodego
Copy link
Member

@stinodego stinodego commented Sep 25, 2023

The new release of pydantic (2.4.0) broke some of the pyiceberg tests.

Not really sure how to properly solve this. I guess the folks over at pyiceberg must release a fix to support the new pydantic version, and/or pydantic must issue a new release to revert the breaking change.

@Fokko Could you track down the issue here?

@github-actions github-actions bot added internal An internal refactor or improvement python Related to Python Polars labels Sep 25, 2023
@stinodego stinodego marked this pull request as ready for review September 25, 2023 17:59
@stinodego stinodego merged commit 5e0dc4d into main Sep 25, 2023
14 checks passed
@stinodego stinodego deleted the pin-pydantic branch September 25, 2023 18:00
@Fokko
Copy link
Contributor

Fokko commented Sep 25, 2023

Hey @stinodego thanks for raising this. I'll dig into this

@Fokko
Copy link
Contributor

Fokko commented Sep 25, 2023

Looks like this has already reported at Pydantic: pydantic/pydantic#7611

@Fokko
Copy link
Contributor

Fokko commented Sep 25, 2023

Thanks for pinging me

romanovacca pushed a commit to romanovacca/polars that referenced this pull request Oct 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal An internal refactor or improvement python Related to Python Polars
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants