+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" <pkara...@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 > maintain...@gluster.org > http://www.gluster.org/mailman/listinfo/maintainers > >
_______________________________________________ Gluster-devel mailing list Gluster-devel@gluster.org http://www.gluster.org/mailman/listinfo/gluster-devel