Open yarikoptic opened 1 year ago
i think this also requires line 302 replaced with the sanitize function from dandi-cli.
Patch coverage: 100.00
% and no project coverage change.
Comparison is base (
b310e3e
) 97.71% compared to head (d22ac9c
) 97.72%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
i think this also requires line 302 replaced with the sanitize function from dandi-cli.
did you try that and it resolved #172? would better be done in a separate PR then since this one does resolve issue too
i think this also requires line 302 replaced with the sanitize function from dandi-cli.
did you try that and it resolved #172? would better be done in a separate PR then since this one does resolve issue too
I tested, it does. Will submit a complimentary PR.
@satra so what do you think about this one? It is complimentary to #175
Also added assertion so we do not count incorrectly. But may be should be just a warning?
Closes #172