From: Harald Nordgren <haraldnordgren@gmail.com>
Date: Wed, 30 Sep 2026 18:40:27 GMT
Subject: Re: [PATCH v3 2/2] ci: point test failures and fixed known breakages at their file and line
Message-ID: <CAHwyqnVt=mSG9u6a_FtuqHDSM+8RVKRNmow7f7MtDvFV4EyOhQ@mail.gmail.com>
In-Reply-To: <5529bccf-eeb1-40f9-ae03-8fa19dc26f5a@gmail.com>

On Wed, Sep 30, 2026 at 4:56 PM Phillip Wood <phillip.wood123@gmail.com> wrote:
>
> Hi Harald
>
> On 30/09/2026 07:09, Harald Nordgren via GitGitGadget wrote:
> > From: Harald Nordgren <haraldnordgren@gmail.com>
> >
> > A test failure or a fixed known breakage gets an annotation that names
> > the test but carries no file or line, so there is nothing to click
> > through to from the GitHub UI.
>
> Have you got an example of this? As I said in my last mail, I can't see
> any links in the output from the linux-leaks job.

That's poor wording on my side, I'll clarify.

> Why do we need to escape the test descriptions when we haven't been
> doing so up to now? Also if the test description is a single line why
> are we worring about '\r'? If it is so important to escape the output
> why does this patch not convert the existing annotations like the
> "group::" on in the trailing context lines?

Yeah, that can be simplified.


Harald