Make Acked/Reviewd/Tested-by tags visible

Jeff Kirsher jeffrey.t.kirsher at intel.com
Thu May 8 09:02:17 EST 2014


On Wed, May 7, 2014 at 1:42 AM, Jeremy Kerr <jk at ozlabs.org> wrote:
> Hi all,
>
>> 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):
>
> OK, I have some progress on this one; but wanted to make sure what I'm
> doing is in the direction that'd be generally useful for you.
>
> How's this look?
>
>   http://ozlabs.org/~jk/tmp/patchwork-ART.png

Looks great!

>
> - there are title attributes on the column header, so you get a tooltip
> saying "Acked-by / Reviewed-by / Tested-by" when hovering over it. The
> actual numbers have specific tooltips too (eg, "2 Acked-by").
>
> Also, are you wanting to filter by the presence of these tags? Just
> non-zero? How about sorting?

It would be beneficial but not crucial.

>
> Anyone have objections to adding a small column to the default patch
> lists? Does anyone want to *not* have this information present?
>

default would be great!

-- 
Cheers,
Jeff


More information about the Patchwork mailing list