dandi / dandisets

730 Dandisets, 807.1 TB total. DataLad super-dataset of all Dandisets from https://github.com/dandisets
10 stars 0 forks source link

Don't run `addurl` until it's needed #351

Closed jwodder closed 1 year ago

jwodder commented 1 year ago

Part of #347.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 56.60% and project coverage change: +0.01 :tada:

Comparison is base (8642675) 76.23% compared to head (8cd2700) 76.24%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## draft #351 +/- ## ========================================== + Coverage 76.23% 76.24% +0.01% ========================================== Files 14 14 Lines 2470 2488 +18 Branches 544 550 +6 ========================================== + Hits 1883 1897 +14 - Misses 435 438 +3 - Partials 152 153 +1 ``` | [Impacted Files](https://app.codecov.io/gh/dandi/dandisets/pull/351?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi) | Coverage Δ | | |---|---|---| | [tools/backups2datalad/adataset.py](https://app.codecov.io/gh/dandi/dandisets/pull/351?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2FkYXRhc2V0LnB5) | `79.72% <27.27%> (ø)` | | | [tools/backups2datalad/asyncer.py](https://app.codecov.io/gh/dandi/dandisets/pull/351?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dandi#diff-dG9vbHMvYmFja3VwczJkYXRhbGFkL2FzeW5jZXIucHk=) | `79.89% <64.28%> (-0.11%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

yarikoptic commented 1 year ago

will pull/cleanup for the next cron to try