codecov-commenter
commented
6 months ago Codecov Report
Attention: Patch coverage is 25.00000% with 3 lines in your changes are missing coverage. Please review.
Project coverage is 94.77%. Comparing base (
1020ea2) to head (423e288).
| Files | Patch % | Lines |
|---|---|---|
| src/math/linear_sieve.rs | 0.00% | 3 Missing :warning: |
Additional details and impacted files
```diff @@ Coverage Diff @@ ## master #718 +/- ## ========================================== - Coverage 94.77% 94.77% -0.01% ========================================== Files 298 298 Lines 22194 22213 +19 ========================================== + Hits 21035 21052 +17 - Misses 1159 1161 +2 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Pull Request Template
Description
@mengfansheng-git discovered in #716 that clippy reports some new warnings. This PR is a quick fix of them.
@siriak please have a look (also on #719).
Checklist:
cargo clippy --all -- -D warningsjust before my last commit and fixed any issue that was found.cargo fmtjust before my last commit.cargo testjust before my last commit and all tests passed.mod.rsfile within its own folder, and in any parent folder(s).DIRECTORY.mdwith the correct link.COUNTRIBUTING.mdand my code follows its guidelines.Please make sure that if there is a test that takes too long to run ( > 300ms), you
#[ignore]that or try to optimize your code or make the test easier to run. We have this rule because we have hundreds of tests to run; If each one of them took 300ms, we would have to wait for a long time.