search

capnproto / capnproto-rust

Cap'n Proto for Rust
MIT License
2.06k stars 222 forks source link

capnp-futures: Use more fine-grained futures-* deps #527

Closed Gelbpunkt closed 1 week ago

Gelbpunkt commented 1 week ago

The large futures crate pulls in a bunch of sub-crates that capnp-futures does not need. We can instead use only the subcrates that are actually required.

codecov[bot] commented 1 week ago

Codecov Report

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

Project coverage is 52.39%. Comparing base (ab342b3) to head (0e5b23c). Report is 111 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #527 +/- ## ========================================== + Coverage 51.64% 52.39% +0.74% ========================================== Files 69 70 +1 Lines 33735 34570 +835 ========================================== + Hits 17422 18112 +690 - Misses 16313 16458 +145 ```

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

dwrensha commented 1 week ago

Looks good to me! (#199 was a previous attempt at this.)