search

liftedinit / many-rs

Rust libraries for the Many Protocol
Apache License 2.0
5 stars 11 forks source link

fix: revert data module serialization (#216) #399

Closed fmorency closed 1 year ago

fmorency commented 1 year ago

Fixes #398

codecov[bot] commented 1 year ago

Codecov Report

Merging #399 (c2abad7) into main (5d62edd) will decrease coverage by 0.15%. The diff coverage is 56.75%.

@@            Coverage Diff             @@
##             main     #399      +/-   ##
==========================================
- Coverage   82.89%   82.75%   -0.15%     
==========================================
  Files         135      135              
  Lines       15756    15502     -254     
==========================================
- Hits        13061    12828     -233     
+ Misses       2695     2674      -21
Impacted Files Coverage Δ
src/many-modules/src/_5_data/get_info.rs 100.00% <ø> (ø)
src/many-modules/src/_5_data/info.rs 100.00% <ø> (ø)
src/many-modules/src/_5_data/query.rs 100.00% <ø> (ø)
src/many-modules/src/lib.rs 82.81% <ø> (ø)
src/many-types/src/cbor.rs 88.88% <0.00%> (+1.21%) :arrow_up:
src/many-types/src/lib.rs 76.81% <ø> (ø)
src/many-modules/src/_5_data/types.rs 28.00% <25.00%> (-62.38%) :arrow_down:
src/many-ledger/src/migration/data.rs 98.41% <100.00%> (ø)
src/many-modules/src/_5_data.rs 84.61% <100.00%> (-0.87%) :arrow_down:

... and 4 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

fmorency commented 1 year ago

I reproduced the issue and the fix using a-vs-b.