<p dir="ltr">+1, Pranith. I prefer adding everyone's name(who reviewed any patch set despite the latest patch set) in the 'Reviewed-by' tag. </p>
<div class="gmail_extra"><br><div class="gmail_quote">On Sep 30, 2016 6:02 PM, "Pranith Kumar Karampuri" <<a href="mailto:pkarampu@redhat.com">pkarampu@redhat.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>hi,<br></div> 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?<br><br></div>Or removing 'Reviewed-by' tag completely would also help to make sure it doesn't give skewed counts.<br clear="all"><div><div><div><br>-- <br><div class="m_4520501216639710159gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Pranith<br></div></div>
</div></div></div></div>
<br>______________________________<wbr>_________________<br>
maintainers mailing list<br>
<a href="mailto:maintainers@gluster.org">maintainers@gluster.org</a><br>
<a href="http://www.gluster.org/mailman/listinfo/maintainers" rel="noreferrer" target="_blank">http://www.gluster.org/<wbr>mailman/listinfo/maintainers</a><br>
<br></blockquote></div></div>