fastapi
🐛 Remove `Required` shadowing from fastapi using Pydantic v2
#12197

Open

Overview Commits 10 Changes View On GitHub

🐛 Remove `Required` shadowing from fastapi using Pydantic v2 #12197

pachewise wants to merge 10 commits into fastapi:master from pachewise:fix/required-shadowing

pachewise commented on 2024-09-13

pachewise force pushed from ba2b2f2b to a6517379 5 days ago

pachewise force pushed from a6517379 to f619617c 3 days ago

pachewise marked this pull request as ready for review 3 days ago

svlandeg added bug

svlandeg changed the title ~~fix: Remove Required shadowing from fastapi using Pydantic v2~~ 🐛 Remove `Required` shadowing from fastapi using Pydantic v2 2 days ago

svlandeg assigned

svlandeg 2 days ago

svlandeg approved these changes on 2024-09-16

svlandeg unassigned

svlandeg 2 days ago

fix: Remove Required shadowing from fastapi using Pydantic v2

3620d690

🎨 [pre-commit.ci] Auto format from pre-commit.com hooks

505d2bf3

change defaults since Ellipsis is not a good one

9d9a674e

revert to using a _compat-friendly Required class

21415e82

🎨 [pre-commit.ci] Auto format from pre-commit.com hooks

299c3438

mypy

9d30a9fa

Update fastapi/_compat.py

3de7c331

PR suggestion - remove commented code

5d65e6d7

add comment for RequiredParam defn when using Pydantic V1

a3752aa8

🎨 [pre-commit.ci] Auto format from pre-commit.com hooks

d7b0eb22

pachewise force pushed from b6b2921e to d7b0eb22 1 day ago

Reviewers

svlandeg

Assignees

No one assigned

Labels

bug

Milestone

No milestone

fastapi 🐛 Remove `Required` shadowing from fastapi using Pydantic v2 #12197 Open

🐛 Remove `Required` shadowing from fastapi using Pydantic v2 #12197

fastapi
🐛 Remove `Required` shadowing from fastapi using Pydantic v2
#12197

Open