Closed SquidDev closed 1 week ago
Attention: Patch coverage is 96.11650%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 84.07%. Comparing base (
8f76041
) to head (be99d2f
). Report is 1 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
src/ser.rs | 90.47% | 0 Missing and 4 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
🚨 Try these New Features:
Sorry, serves me right for trying to make "tiny" changes just before going to bed! Have fixed that.
All good; thanks very much for this!
This updates Pythonize to work with PyO3 0.23.x. Apologies if you've started work on this already — feel free to close if so :).
xyz_bound
methods have been renamed toxyz
.IntoPyObject
for conversion to Python. Several of the conversion methods now return aBorrowed
rather thanBound
value, and so some of the serialisation functions need to convert to aBound<_>
first — I've moved that into a helper function, but happy to inline that if preferred.eval
/run
now take a&CStr
rather than&str
, so we use C string literals in the tests.