<div dir="ltr">If you want it to happen automatically, it's next to impossible. We'll need to fork Gerrit to make those changes.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 30, 2016 at 6:08 PM, Manikandan Selvaganesh <span dir="ltr"><<a href="mailto:manikandancs333@gmail.com" target="_blank">manikandancs333@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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"><div><div class="h5">On Sep 30, 2016 6:02 PM, "Pranith Kumar Karampuri" <<a href="mailto:pkarampu@redhat.com" target="_blank">pkarampu@redhat.com</a>> wrote:<br type="attribution"></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><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 data-smartmail="gmail_signature"><div dir="ltr">Pranith<br></div></div>
</div></div></div></div>
<br></div></div>______________________________<wbr>_________________<br>
maintainers mailing list<br>
<a href="mailto:maintainers@gluster.org" target="_blank">maintainers@gluster.org</a><br>
<a href="http://www.gluster.org/mailman/listinfo/maintainers" rel="noreferrer" target="_blank">http://www.gluster.org/mailman<wbr>/listinfo/maintainers</a><br>
<br></blockquote></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><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">nigelb<br></div></div>
</div>