Re: b4 review tui: too smart reply features?
Matthieu Baerts <[email protected]> Tue, 30 Jun 2026 12:55:25 +0200
| Newsgroups | org.kernel.linux.tools |
|---|---|
| Organization | NGI0 Core |
| Message-ID | <[email protected]> |
Hi Konstantin, On 11/06/2026 03:56, Konstantin Ryabitsev wrote: > On Tue, Apr 28, 2026 at 12:52:01PM +0200, Matthieu Baerts wrote: >> Hi Konstantin, >> >> I'm still trying 'b4 review tui', and something that bothers me is when >> my replies are mangled. I have three examples. > > Okay, the latest master has revamped that quite a bit. I was also not really > happy with the way I kept fighting with the tooling, which is the opposite of > where I want to go. I had the bright idea to move some of the functionality > into a vim plugin instead of doing it in b4 directly -- this way you still get > most of the neat features but without the inability to undo or forced/awkward > quote trimming. > > Can you try it out and see if you're fighting with the current master less? Thank you for the update, and sorry for the delay! I just retried, and it is indeed much better now! Yet, I still have an issue with trailers, e.g. Please add a Fixes tag. Here I guess it should be: Fixes: (...) The Review view then correctly showed my comment with the Fixes tag, but *I think* the Fixes tag was not visible in green at the bottom left (not sure, because I also added a Reviewed-by tag after the reply). When looking at the 'email mode' view, the Fixes tag got dropped, leaving 2 empty lines around. Back to the 'reply' view, vim showed all lines ending with ^M, and the line starting with 'Fixes:' was no longer there. Adding it back re-added it to the other views, and the Fixes tag was also listed at the bottom left in the 'review' view, but I could see extra new lines due to all these \r. Stripping them from the 'reply' view fixes all the issues, and I was able to send the reply with both my Fixes and Reviewed-by tags. Any idea was caused that? :) BTW, after having sent a reply, it appears a few seconds later as unread in my inbox. I guess only a filter on the inbox side can help to mark them as read, right? Cheers, Matt -- Sponsored by the NGI0 Core fund.