search

uutils / findutils

Rust implementation of findutils
MIT License
280 stars 36 forks source link

xargs: Implement replace / -I (Closes: #310) #323

Closed sylvestre closed 3 months ago

sylvestre commented 4 months ago

Closes: #310

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 78.07018% with 25 lines in your changes are missing coverage. Please review.

Project coverage is 58.36%. Comparing base (274d749) to head (45c1938).

Files Patch % Lines
tests/xargs_tests.rs 72.22% 0 Missing and 20 partials :warning:
src/xargs/mod.rs 88.09% 2 Missing and 3 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #323 +/- ## ========================================== + Coverage 58.12% 58.36% +0.23% ========================================== Files 30 30 Lines 3630 3713 +83 Branches 821 841 +20 ========================================== + Hits 2110 2167 +57 - Misses 1220 1223 +3 - Partials 300 323 +23 ```

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

cakebaker commented 3 months ago

Good work :)

sylvestre commented 3 months ago

many thanks for this terrific review :)