[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/maintainers/attachments/20160930/f84d77e4/attachment.html>
More information about the maintainers
mailing list