Skip to content

Remove redundant cbor encoding - #7644

Closed
sudo-shashank wants to merge 4 commits into
mainfrom
shashank/redundant-cbor-encoding
Closed

sudo-shashank wants to merge 4 commits into
mainfrom
shashank/redundant-cbor-encoding

cleanup

ca97f99
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 22, 2026 in 0s

67.19% (+0.01%) compared to 41414f0

View this Pull Request on Codecov

67.19% (+0.01%) compared to 41414f0

Details

Codecov Report

❌ Patch coverage is 87.50000% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.19%. Comparing base (41414f0) to head (ca97f99).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/chain_sync/tipset_syncer.rs 75.00% 2 Missing and 2 partials ⚠️
src/utils/encoding/mod.rs 86.36% 1 Missing and 2 partials ⚠️
src/utils/cid/mod.rs 93.75% 0 Missing and 1 partial ⚠️
Additional details and impacted files
Files with missing lines Coverage Δ
src/chain_sync/validation.rs 88.96% <100.00%> (+0.18%) ⬆️
src/shim/crypto.rs 90.90% <100.00%> (ø)
src/utils/cid/mod.rs 93.93% <93.75%> (-0.18%) ⬇️
src/utils/encoding/mod.rs 96.47% <86.36%> (+2.27%) ⬆️
src/chain_sync/tipset_syncer.rs 65.74% <75.00%> (+0.74%) ⬆️

... and 6 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5bb5ce...ca97f99. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.