search

dandi / dandisets-healthstatus

Healthchecks of dandisets and support libraries (pynwb and matnwb)
0 stars 1 forks source link

Move shell scripts to `tools/` #68

Closed jwodder closed 8 months ago

jwodder commented 8 months ago

The script is not part of the dandisets-healthstatus package and thus does not belong in code/.

codecov[bot] commented 8 months ago

Codecov Report

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

Comparison is base (f362c2f) 61.58% compared to head (b7aa6c0) 61.58%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #68 +/- ## ======================================= Coverage 61.58% 61.58% ======================================= Files 9 9 Lines 669 669 Branches 164 164 ======================================= Hits 412 412 Misses 238 238 Partials 19 19 ```

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

yarikoptic commented 8 months ago

Agree -- better not in code/. Let's move under tools/ or alike then -- better not to trash the top level.

jwodder commented 8 months ago

@yarikoptic Should run.sh and run_loop.sh also be moved to tools/?

yarikoptic commented 8 months ago

I think so since they are just to help running this one but they might need tune ups to remain working

jwodder commented 8 months ago

@yarikoptic Should this be merged now?