[openstack-dev] Missing column in Gerrit overview page showing a -1 review after/with a +2 review

Romil Gupta romilgupta19 at gmail.com
Sat Feb 14 12:12:31 UTC 2015


Awesome, I like the idea.

On Sat, Feb 14, 2015 at 3:18 PM, Christian Berendt <christian at berendt.io>
wrote:

> At the moment there are three columns in the Gerrit overiew page showing
> the status of a review request:
>
> * CR (Code-Review)
> * V (Verified)
> * Workflow (WF).
>
> After a core reviewer +2 (CR) a review request there will be a green
> hook in the CR column in the overview page, regardless of -1 reviews.
>
> Sometimes I do not have the time to have a look in my Gerrit mail
> folder. Sometimes I do not have the time to manually monitor pending
> review requests. This is why I sometimes miss a new -1 of a reviewer
> because it is not shown in the Gerrit overview page when there is
> already a +2 of a core reviewer. Also a +2 of a core reviewer will
> "overwrite" an existing -1 in the Gerrit overview page. Sometimes I do
> not have the time to remove my +2 after a new -1 (at the moment it is
> necessary to remove a +2 after a new -1 to be able to see the new -1 on
> the Gerrit overview page).
>
> I would really like to have a fourth column in the Gerrit overview page
> showing a -1 on a review request even if there is already a +2 of a core
> reviewer.
>
> What do you think about this?
>
> And if this idea is appreciated how can this fourth column be added to
> the Gerrit overview page?
>
> Christian.
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>



-- 
*Regards,*

*Romil *
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20150214/e8f75fc0/attachment.html>


More information about the OpenStack-dev mailing list