Skip to content

perf(bgzf): skip BGZF CRC verification on trusted stdin input by default (#786) - #798

Merged
nh13 merged 4 commits into
mainfrom
786/nhomer/perf-bgzf-crc-skip
Aug 20, 2026
Merged

nh13 merged 4 commits into
mainfrom
786/nhomer/perf-bgzf-crc-skip

docs(bam-io): document PipelineReaderOpts.verify_crc as pending the f…

68dbde5
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 19, 2026 in 0s

94.41% (+0.01%) compared to 4ac5c2c

View this Pull Request on Codecov

94.41% (+0.01%) compared to 4ac5c2c

Details

Codecov Report

❌ Patch coverage is 95.59543% with 27 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.41%. Comparing base (4ac5c2c) to head (68dbde5).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/lib/unified_pipeline/base.rs 12.50% 14 Missing ⚠️
src/lib/commands/common.rs 81.81% 8 Missing ⚠️
src/lib/unified_pipeline/fastq.rs 71.42% 4 Missing ⚠️
src/lib/commands/clip.rs 98.79% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #798      +/-   ##
==========================================
+ Coverage   94.40%   94.41%   +0.01%     
==========================================
  Files         186      186              
  Lines      114196   114772     +576     
==========================================
+ Hits       107806   108364     +558     
- Misses       6390     6408      +18     

☔ 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.