+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@xxxxxxxxxx> wrote:
Or removing 'Reviewed-by' tag completely would also help to make sure it doesn't give skewed counts.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?
--Pranith
_______________________________________________
maintainers mailing list
maintainers@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/maintainers
_______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://www.gluster.org/mailman/listinfo/gluster-devel
