Skip to content

refactor: replace all unwrap() calls in src/lib/ with expect() - #226

Merged
nh13 merged 1 commit into
mainfrom
nh/unwrap-audit-lib
Apr 4, 2026
Merged

nh13 merged 1 commit into
mainfrom
nh/unwrap-audit-lib

refactor: replace all unwrap() calls in src/lib/ with expect()

5d5ef2d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Apr 4, 2026 in 0s

88.97% (+0.09%) compared to 0ee001f

View this Pull Request on Codecov

88.97% (+0.09%) compared to 0ee001f

Details

Codecov Report

❌ Patch coverage is 87.29097% with 114 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.97%. Comparing base (0ee001f) to head (5d5ef2d).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/lib/unified_pipeline/base.rs 3.63% 106 Missing ⚠️
src/lib/sort/keys.rs 66.66% 4 Missing ⚠️
src/lib/sort/raw.rs 90.90% 3 Missing ⚠️
src/lib/batched_sam_reader.rs 95.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #226      +/-   ##
==========================================
+ Coverage   88.88%   88.97%   +0.09%     
==========================================
  Files         113      113              
  Lines       53521    54107     +586     
==========================================
+ Hits        47574    48144     +570     
- Misses       5947     5963      +16     

☔ View full report in Codecov by Sentry.
📢 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.