patchwork icon indicating copy to clipboard operation
patchwork copied to clipboard

Add a way to a list of all patches with some set of A/R/T tags

Open dlespiau opened this issue 10 years ago • 1 comments

Right now we can't filter/sort the patches using tags. Sounds like a very useful thing to have to answer questions like "what are the reviewed patches that are, then, good candidates for merging"?

dlespiau avatar Sep 30 '15 10:09 dlespiau

Sorry, I have tons of questions: bear with me. For patches, there are now 3 tags that I can see: Acked-by, Reviewed by and Tested by.

Regarding reviewed by, shouldn't patches with this tag change status to reviewed? Or should that only happen when the assigned reviewer sends a 'Reviewed-by:' to the mailing list?

In the case of the tested by tag. I understand this is set when someone sends a "Tested-by:" to the mailing list. But what about automated tests? If a patch is included in some kind of automated testing run, and the results of such testing are shipped to Patchwork, should the patch get a +1 in its 'tested by' tag? Or are automated test results only sent for series, never for single patches?

Regarding the querying you mention: what combination of tags indicates a good candidate for merging? Those that are reviewed and tested but at least one person? By a specific person, e.g. the assigned reviewer?

Thanks!

belenbarrospena avatar Nov 27 '15 12:11 belenbarrospena