Make Acked/Reviewd/Tested-by tags visible
Jeremy Kerr
jk at ozlabs.org
Tue Nov 12 11:37:26 EST 2013
Hi Yann,
> We would like to suggest that the web GUI and the pwclient CLI both
> display such tags besides each patch, a bit like (hypotetical output of
> pwclient):
>
> ID Tags State Subject
> 123456 ART New Wonderfull patch to apply quickly
I've been looking to implement this for a while, but didn't have any
good ideas about how to present this in the UI. I love your proposal
here with the flags, it's simple and I think would work well for the
majority of cases.
> Since patchwork already inserts those tags to the commit log (eg.
> pwclient view PATCH-ID), I hope it would be easy enough to also display
> it in the patch list.
It's easy enough to pull these out of the comment at patch-display time,
but a little more tricky with list views (without some horrible SQL
joining). I'll work out a nice way to to this; I think it'll involve
applying these flags when the follow-up comment is parsed.
Other projects have different policies for their tagging (some require 3
acks, for example), which will introduce a little complexity there too,
but that's not insurmountable either.
Cheers,
Jeremy
More information about the Patchwork
mailing list