search

uutils / findutils

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

find: Changed exit code when an error is encountered in `process_dir()`. #352

Closed hanbings closed 3 months ago

hanbings commented 3 months ago

fix: https://github.com/uutils/findutils/issues/27

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 59.52%. Comparing base (d94a8e9) to head (4d05816).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #352 +/- ## ========================================== + Coverage 59.33% 59.52% +0.18% ========================================== Files 30 30 Lines 3819 3822 +3 Branches 869 869 ========================================== + Hits 2266 2275 +9 + Misses 1221 1218 -3 + Partials 332 329 -3 ```

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

cakebaker commented 3 months ago

Can you please add a test to ensure we don't regress in the future?

hanbings commented 3 months ago

Can you please add a test to ensure we don't regress in the future?

OK, I've added a test for this fix.

cakebaker commented 3 months ago

Good work, thanks :)