Skip to content

perf(compare): make compare bams faster than the sort it validates - #696

Merged
nh13 merged 7 commits into
mainfrom
686/nh/perf-compare-bams
Aug 6, 2026
Merged

nh13 merged 7 commits into
mainfrom
686/nh/perf-compare-bams

test(compare): pin the sort-order contract and cover the multithreade…

2b68b78
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 6, 2026 in 0s

94.06% (+0.13%) compared to c09669a

View this Pull Request on Codecov

94.06% (+0.13%) compared to c09669a

Details

Codecov Report

❌ Patch coverage is 97.75641% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.06%. Comparing base (c09669a) to head (2b68b78).
⚠️ Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
src/lib/commands/compare/engines/positional.rs 87.23% 6 Missing ⚠️
src/lib/commands/compare/engines/sort_verify.rs 98.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #696      +/-   ##
==========================================
+ Coverage   93.93%   94.06%   +0.13%     
==========================================
  Files         178      178              
  Lines      108128   108601     +473     
==========================================
+ Hits       101565   102152     +587     
+ Misses       6563     6449     -114     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.