adding docstrings #795
adding docstrings #795
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #795 +/- ##
==========================================
- Coverage 93.52% 93.48% -0.04%
==========================================
Files 100 102 +2
Lines 14379 14606 +227
Branches 25 25
==========================================
+ Hits 13448 13655 +207
- Misses 925 945 +20
Partials 6 6
... and 11 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
CodSpeed Performance ReportMerging #795 will not alter performanceComparing Summary
Benchmarks breakdown
|
| include: A set of fields to include, if `None` all fields are included. | ||
| exclude: A set of fields to exclude, if `None` no fields are excluded. |
There was a problem hiding this comment.
The types of include and exclude are _IncEx - we may want to make that alias public and document it.
|
|
||
| @property | ||
| def value(self) -> _T: ... | ||
| def value(self) -> _T: |
There was a problem hiding this comment.
The type annotation is _T, maybe we should rename to T for nicer docs?
| def __new__(cls, schema: CoreSchema, config: CoreConfig | None = None) -> Self: | ||
| """ | ||
| Create a new SchemaValidator. | ||
| Arguments: | ||
| schema: The [`CoreSchema`][pydantic_core.core_schema.CoreSchema] to use for validation. | ||
| config: Optionally a [`CoreConfig`][pydantic_core.core_schema.CoreConfig] to configure validation. | ||
| """ |
There was a problem hiding this comment.
It looks like __new__ doesn't show in the docs. @Kludex also reported an issue with __new__ and __init__ mismatching arguments in some cases (e.g. for exceptions, __init__ comes from BaseException). I think this is probably a PyO3 bug, PyO3 sets __new__ but not __init__.
Maybe for now it's more practical to set both __new__ and __init__ here, document __init__, and I'll have a think about how to resolve this in PyO3?
|
going to merge this in the hope of including it in the next release, I'll fix those things in a new PR. |
Original-commit-hash: 7802821
2 participants
WIP