dandi / dandisets-healthstatus

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

Make status"es" into hyperlinks pointing to files for that dandiset #12

Closed yarikoptic closed 1 year ago

yarikoptic commented 1 year ago

E.g. in the line

000362 | 52 passed, 0 failed, 0 timed out | 0 passed, 2 failed, 50 timed out

the last column would be (besides references relative, not absolute URLs like here):

000362 | 52 passed, 0 failed, 0 timed out | 0 passed, 2 failed, 50 timed out

I do realize that getting to corresponding line numer might be tricky but solutions in https://github.com/yaml/pyyaml/issues/456 suggest it might be possible?

if not possible in some sensible fashion, at least make it urls to that status.yaml so it is still reasonably easy to get to those files.

jwodder commented 1 year ago

@yarikoptic I don't think linking to individual lines is reasonably feasible. Thus, we'd instead be linking both columns (I'm assuming you're just focusing on the last column here for illustration purposes and that you want the second column to be a link as well) to {dandiset_id}/status.yaml, at which point it seems more reasonable to solely link the Dandiset ID in the first column to the status.yaml file, i.e:

000362 | 52 passed, 0 failed, 0 timed out | 0 passed, 2 failed, 50 timed out

Is that acceptable?

yarikoptic commented 1 year ago

ok.