Re: b4 review available in master

Conor Dooley
2026-02-28T22:16:18Z

[-- Attachment #1: Type: text/plain, Size: 1276 bytes --]

On Sat, Feb 28, 2026 at 12:34:40PM -0500, Konstantin Ryabitsev wrote:
> On Sat, Feb 28, 2026 at 12:21:27PM -0500, Konstantin Ryabitsev wrote:
> > Argh, escape_markup will, unhelpfully, only escape actual markup. So, it will
> > escape [bold] into \[bold\] but leave [PATCH] as [PATCH]. WTF, who decided
> > this was a good idea?
> >
> > I'll commit a more wholesome fix shortly, sorry about that.
>
> OK, the more wholesome fix is in place now. Please try it out and hopefully
> that lifts the curse.

Yeah, got me further but I got stuck again - although not on a crash. I
sent you a mail with a screenshot of where I am :)

One thing I noticed was missing was that I have no idea while looking at
a patch who sent it. If you look at the mail I sent with the screenshot:
Lv Zheng is the submitter, so the signoff chain is wrong (cos there's no
from header in the patch to make Jingyu the author, so either there's a
co-developed-by missing or a from header is needed), but it's impossible
to tell from this UI. Once you hit "reply", then you can see it, but not
from the main view.

I have some other thoughts that are half baked, and I will try to
elaborate on them once I have at least tried to do a days review with
the tool.

Cheers,
Conor.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]