[Gluster-Maintainers] 'Reviewd-by' tag for commits

Manikandan Selvaganesh manikandancs333 at gmail.com
Fri Sep 30 12:38:33 UTC 2016


+1, Pranith. I prefer adding everyone's name(who reviewed any patch set
despite the latest patch set) in the 'Reviewed-by' tag.

On Sep 30, 2016 6:02 PM, "Pranith Kumar Karampuri" <pkarampu at redhat.com>
wrote:

> hi,
>      At the moment 'Reviewed-by' tag comes only if a +1 is given on the
> final version of the patch. But for most of the patches, different people
> would spend time on different versions making the patch better, they may
> not get time to do the review for every version of the patch. Is it
> possible to change the gerrit script to add 'Reviewed-by' for all the
> people who participated in the review?
>
> Or removing 'Reviewed-by' tag completely would also help to make sure it
> doesn't give skewed counts.
>
> --
> Pranith
>
> _______________________________________________
> maintainers mailing list
> maintainers at gluster.org
> http://www.gluster.org/mailman/listinfo/maintainers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.gluster.org/pipermail/maintainers/attachments/20160930/672785ba/attachment.html>


More information about the maintainers mailing list