search

davidhewitt / pythonize

MIT License
198 stars 28 forks source link

Expose the Pythonizer and Depythonizer to integrate with the serde_path_to_err crate #52

Closed Humandoodlebug closed 7 months ago

Humandoodlebug commented 7 months ago

Since #31 seems inactive, this PR extends it with tests as requested by @davidhewitt.

Credit to @sandhose for the original commit, I've just added tests on top.

Fixes #30

codecov-commenter commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (21a911c) 79.02% compared to head (f840ea7) 80.96%.

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

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #52 +/- ## ========================================== + Coverage 79.02% 80.96% +1.93% ========================================== Files 3 3 Lines 1068 1077 +9 Branches 1068 1077 +9 ========================================== + Hits 844 872 +28 + Misses 161 144 -17 + Partials 63 61 -2 ```

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

davidhewitt commented 7 months ago

Thanks for picking this up! Looks good to me!

The clippy failures are unrelated, I'll tidy them up in a separate PR.