-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Add runtime validation of @validate_call
callable argument
#10627
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
Deploying pydantic-docs with
|
Latest commit: |
cc74d42
|
Status: | ✅ Deploy successful! |
Preview URL: | https://bb03277e.pydantic-docs.pages.dev |
Branch Preview URL: | https://validate-call-runtime-tc.pydantic-docs.pages.dev |
CodSpeed Performance ReportMerging #10627 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.
'__name__', | ||
'__qualname__', | ||
'__annotations__', | ||
'__dict__', # required for __module__ |
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.
Have we thought about the consequences of removing this?
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.
__name__
was necessary for the core_config title thing but #10562 made it not necessary anymore. For the rest, I don't think it will change anything.
Didn't realize things were set to auto merge, let's still chat about the above questions + potentially address them in a new small PR. |
Change Summary
Wrapping up #10484
Closes #10461
Related issue number
Checklist