search

JakobGM / patito

A data modelling layer built on top of polars and pydantic
MIT License
252 stars 23 forks source link

Fix some warnings after migration #39

Closed thomasaarholt closed 5 months ago

thomasaarholt commented 5 months ago

I'm fixing some of the warnings introduced with the pydantic PR. Mainly adding docstrings where we had duplicates to copy from elsewhere, but also some minor typing things.

codecov-commenter commented 5 months ago

Codecov Report

Attention: Patch coverage is 91.66667% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 85.78%. Comparing base (5446910) to head (40de7e2).

Files Patch % Lines
src/patito/pydantic.py 90.00% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #39 +/- ## ========================================== + Coverage 85.76% 85.78% +0.01% ========================================== Files 12 12 Lines 1075 1076 +1 Branches 286 286 ========================================== + Hits 922 923 +1 Misses 94 94 Partials 59 59 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.