-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Support initializing root model fields with values of the root
type in the mypy plugin
#11212
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
Conversation
… in the mypy plugin Also use the `TypeInfo.has_base` method instead of iterating over the MRO in various places.
Deploying pydantic-docs with
|
Latest commit: |
9dd8b0a
|
Status: | ✅ Deploy successful! |
Preview URL: | https://37172f3c.pydantic-docs.pages.dev |
Branch Preview URL: | https://root-model-plugin.pydantic-docs.pages.dev |
CodSpeed Performance ReportMerging #11212 will not alter performanceComparing Summary
|
Coverage reportClick to see where and how coverage changed
This report was generated by python-coverage-comment-action |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This LGTM overall (thanks for adding the tests, those made the impact here quite clear), but let's have @dmontagu take a look as well given his familiarity with the mypy
plugin.
Hello! We have started noticing these errors:
Can this change be the cause of these new error? It was working fine last week. |
Indeed, rolling back to 2.10.6 and this problem does not appear. So I guess this has pretty much broken our mypy checks :( |
@antoniogamizdelgado please open a new issue with a MRE. |
Also use the
TypeInfo.has_base
method instead of iterating over the MRO in various places.Closes #11203.
Change Summary
Related issue number
Checklist