FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

⬆️ Bump Pydantic to 2.13.4 by YuriiMotov · Pull Request #1985 · fastapi/sqlmodel · GitHub

Repository navigation

⬆️ Bump Pydantic to 2.13.4 - #1985

Merged
svlandeg merged 6 commits into
mainfrom
bump-pydantic-2.13
Jun 4, 2026
Merged

svlandeg merged 6 commits into
mainfrom
bump-pydantic-2.13

Conversation

YuriiMotov commented May 27, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Description

Pydantic 2.13 added polymorphic_serialization parameter to model_dump and linting started to fail with Liskov substitution principle violation error.

I added polymorphic_serialization to SQLMOdel's model_dump and added test for it

AI Disclaimer

No AI used

Checklist

  • This PR is an obvious typo fix, or it links to a GitHub Discussion for the proposed code change.
  • I added tests for the change.
  • The new or updated tests fail on the main branch and pass on this PR.
  • Coverage stays at 100%. Note: the Coverage is 99.00% for this project
  • The documentation explains the change if needed.

YuriiMotov added dependencies Pull requests that update a dependency file internal Internal changes python:uv Pull requests that update python:uv code labels May 27, 2026

Copy link
Copy Markdown
Contributor

📝 Docs preview

Last commit e5b9812 at: https://d00d935c.sqlmodel.pages.dev

svlandeg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM!

svlandeg merged commit 55265b5 into main Jun 4, 2026
36 checks passed
svlandeg deleted the bump-pydantic-2.13 branch June 4, 2026 07:52
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file internal Internal changes python:uv Pull requests that update python:uv code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL