search

uutils / findutils

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

find: Implement `-anewer` and `-cnewer`. #386

Closed hanbings closed 1 month ago

hanbings commented 1 month ago

implement: https://github.com/uutils/findutils/issues/370

Added extra judgment to parse_str_to_newer_args() function.

  1. -anewer is equivalent to -neweram
  2. -cnewer is equivalent to -newercm
  3. -newer is equicalent to -newermm

BFS test +1 PASS in commit 57f2a3bc0751f617933f551a531bdf45acdf77fd.

This PR does not solve Parsing multiple time formats, I will open a new PR to handle it separately. : )

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 59.04%. Comparing base (92c6059) to head (fc02681). Report is 8 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #386 +/- ## ========================================== + Coverage 58.70% 59.04% +0.33% ========================================== Files 30 30 Lines 3821 3855 +34 Branches 841 850 +9 ========================================== + Hits 2243 2276 +33 - Misses 1254 1255 +1 Partials 324 324 ```

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

hanbings commented 1 month ago

Thanks. I'm sorry for these issues, they have been fixed in new commits.

cakebaker commented 1 month ago

Thanks. There's no need to be sorry, the things I mentioned are minor details ;-) And it's great that you fixed them in time.rs, too.