Re: [PATCH v5 0/2] ci: link failure and leak annotations to the test script

>>> Is this enough to call this a regression? Then maybe it's not worth
>>> doing this part at all.
>>
>> Yes, I think we should drop this patch. The first step to debugging a
>> test failure is to look at the test output, so the current behavior
>> where clicking on the links on the summary page takes you to the test
>> output is more useful than taking you to a diff that may not even show
>> the test that failed. The first patch is definitely worth keeping as it
>> makes it much easier to see the LSAN output.
> > I played with instead showing the file name (and line when available)
> as part of the annotation text, and leaving the linking as it is. I
> think it could gives us the best of both worlds:
> >      memory leak logged in t1060 (t1060-object-corruption.sh)
> > and
> >      failed: t1060.17 partial clone of corrupted repository
> (t1060-object-corruption.sh:141)
> > What do you think?