Skip to content

address review: type_name in default error reasons + warnings, tighte…

f28cb15
Select commit
Loading
Failed to load commit list.
Merged

refactor(strategies,position): remove todo!()/unimplemented!() (#322) #356

address review: type_name in default error reasons + warnings, tighte…
f28cb15
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Apr 18, 2026 in 1s

76.24% (target 72.00%)

View this Pull Request on Codecov

76.24% (target 72.00%)

Details

Codecov Report

❌ Patch coverage is 11.26761% with 63 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/strategies/base.rs 1.96% 50 Missing ⚠️
src/strategies/long_put.rs 0.00% 4 Missing ⚠️
src/strategies/short_call.rs 0.00% 4 Missing ⚠️
src/strategies/short_put.rs 0.00% 4 Missing ⚠️
src/strategies/long_call.rs 75.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
src/model/position.rs 82.97% <100.00%> (+0.84%) ⬆️
src/strategies/long_call.rs 71.25% <75.00%> (+0.36%) ⬆️
src/strategies/long_put.rs 70.76% <0.00%> (-0.89%) ⬇️
src/strategies/short_call.rs 73.11% <0.00%> (-0.90%) ⬇️
src/strategies/short_put.rs 71.47% <0.00%> (-0.89%) ⬇️
src/strategies/base.rs 52.53% <1.96%> (-9.74%) ⬇️

... and 1 file with indirect coverage changes

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