search

nextest-rs / nextest

A next-generation test runner for Rust.
https://nexte.st
Apache License 2.0
2.22k stars 99 forks source link

[nextest-runner] tackle some of the too_many_arguments clippy lints #1837

Closed sunshowers closed 3 weeks ago

sunshowers commented 3 weeks ago

Add some wrapper structs to pass in arguments. We're going to be adding even more arguments in the future, and these wrappers make that job significantly easier.

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 80.59701% with 13 lines in your changes missing coverage. Please review.

Project coverage is 79.74%. Comparing base (6bbe40d) to head (1eafe2e). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
nextest-runner/src/runner.rs 77.96% 13 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1837 +/- ## ========================================== - Coverage 79.79% 79.74% -0.06% ========================================== Files 80 80 Lines 20375 20363 -12 ========================================== - Hits 16259 16239 -20 - Misses 4116 4124 +8 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.