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.

Sign up for GitHub

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

fix: pydantic 2.10.x breaking change #1095

Merged
merged 8 commits into from
Jan 2, 2025
Merged

fix: pydantic 2.10.x breaking change #1095

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
4f423b6
fix: breaking change with pydantic 2.10.x
mdaffad Dec 24, 2024
fc2b38a
chore: test on gha
mdaffad Dec 24, 2024
ee11cd9
fix: unify python and json validator schema
mdaffad Dec 27, 2024
b045335
Merge branch 'main' of github.com:mdaffad/beanie into fix-pydantic-2.…
mdaffad Dec 27, 2024
848d14c
fix: revert version checking logic
mdaffad Dec 28, 2024
ce4ab1f
fix: drop on push event on github actions
mdaffad Dec 28, 2024
ae0d0a7
fix: marker logic for pydantic v2.10
mdaffad Dec 28, 2024
04bdc8d
fix: add test pydantic 2.10.4
mdaffad Dec 28, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 4 additions & 14 deletions beanie/odm/utils/pydantic.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,22 +1,12 @@
from typing import Any, Type

import pydantic
from packaging.specifiers import SpecifierSet
from packaging.version import Version
from pydantic import BaseModel


def is_version_valid(version, requirement):
# Parse the requirement as a SpecifierSet
specifiers = SpecifierSet(requirement)
# Parse the version as a Version
v = Version(version)
# Check if the version satisfies the specifiers
return v in specifiers


IS_PYDANTIC_V2 = is_version_valid(pydantic.VERSION, ">=2")
IS_PYDANTIC_V2_10 = is_version_valid(pydantic.VERSION, ">=2.10")
IS_PYDANTIC_V2 = int(pydantic.VERSION.split(".")[0]) >= 2
IS_PYDANTIC_V2_10 = (
IS_PYDANTIC_V2 and int(pydantic.VERSION.split(".")[1]) >= 10
) or int(pydantic.VERSION.split(".")[0]) > 2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would remove this "or" part. Any other major Pydantic version would not necessarily be compatible with Beanie and I think we should not bet on this.
Lets make this check do what it says in the name, to explicitly check for Pydantic v2.10.x (or above, constrained to v2).


if IS_PYDANTIC_V2:
from pydantic import TypeAdapter
Expand Down