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

Nigel Babu nigelb at redhat.com
Fri Sep 30 12:45:28 UTC 2016


If you want it to happen automatically, it's next to impossible. We'll need
to fork Gerrit to make those changes.

On Fri, Sep 30, 2016 at 6:08 PM, Manikandan Selvaganesh <
manikandancs333 at gmail.com> wrote:

> +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
>>
>>
> _______________________________________________
> maintainers mailing list
> maintainers at gluster.org
> http://www.gluster.org/mailman/listinfo/maintainers
>
>


-- 
nigelb
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.gluster.org/pipermail/gluster-devel/attachments/20160930/f84d77e4/attachment.html>


More information about the Gluster-devel mailing list