-
-
Notifications
You must be signed in to change notification settings - Fork 7.4k
🐛 Propagate Pydantic's model config #12504
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
base: master
Are you sure you want to change the base?
Conversation
I haven't checked this thoroughly but I suspect it should be fixed in Pydantic in I haven't played with it locally yet, running the code, etc. I might be wrong. |
Yep, you are correct. It seems that the problem is more complex than I initially thought. So yeah, my PR does not work. I added to this PR test case that should pass |
Is this issue still open? Looking to contribute |
@shrey1605 yep, issue is still open |
Related discussion:
#12503