Dear Jeremy Kerr, On Wed, 11 Jun 2014 10:00:54 +1000, Jeremy Kerr wrote:
> They should be parsed when the original patch is received, and when any > follow-ups are appended to the patch. However: > > > I noticed that (in the Buildroot patchwork [0]) some tags were not > > accounted for: > > http://patchwork.ozlabs.org/patch/354654/ (missing reviewed-by) > > http://patchwork.ozlabs.org/patch/354590/ (missing acked-by) > > [...] > > .. it looks like I may be missing a case there. I'll take a look. Another thing that is missing IMO is a way of sorting the patches by the number of A/R/T tags. Probably not needed to be able to sort them for each individual value, but being able to sort them by the sum of A+R+T would be useful so that maintainers can spot immediately which patches have received Acked-by, Tested-by or Reviewed-by tags. Thanks for this very useful new feature! Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork