search

psyinfra / onyo

text-based inventory system on top of git
ISC License
3 stars 5 forks source link

BF/DOC: unify syntax of `OnyoRepo` and `GitRepo` `.clear_cache()` #610

Closed aqw closed 3 months ago

aqw commented 3 months ago

I noticed these while looking into #322

This unifies the approach, and updates the doc strings.

codecov[bot] commented 3 months ago

Codecov Report

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

Project coverage is 95.76%. Comparing base (da03fc8) to head (f65e934). Report is 7 commits behind head on main.

Files Patch % Lines
onyo/commands/tests/test_set.py 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #610 +/- ## ========================================== + Coverage 95.70% 95.76% +0.06% ========================================== Files 71 69 -2 Lines 5608 5621 +13 Branches 1097 1099 +2 ========================================== + Hits 5367 5383 +16 + Misses 141 139 -2 + Partials 100 99 -1 ```

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

TobiasKadelka commented 3 months ago

Thanks, looks good to me; I will merge! :)