Remove default module globals from types namespace#10123
Conversation
|
On this branch: On |
|
So this doesn't make a huge difference in terms of memory usage, but it makes debugging and namespace management cleaner / easier. |
CodSpeed Performance ReportMerging #10123 will not alter performanceComparing Summary
|
Coverage reportClick to see where and how coverage changed
This report was generated by python-coverage-comment-action |
||||||||||||||||||||||||
Deploying pydantic-docs with
|
| Latest commit: |
aa8b29d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://6d56da38.pydantic-docs.pages.dev |
| Branch Preview URL: | https://ns-cluttering.pydantic-docs.pages.dev |
|
I've added |
dmontagu
left a comment
There was a problem hiding this comment.
LGTM, though we should keep an eye out for anything this change might break, wouldn't surprise me if there were some exotic scenarios that might be affected, but it seems to me that worst case you will just need to manually rebuild, so I think it should be okay.
On `main` right now, when we store or fetch namespaces for classes, we get a mess as shown below
This takes up lots of unneeded space, and we don't use these global defaults for type resolution or schema building, so there's no reason for us to store them.
Makes progress on #10074