[openstack-dev] *How* to keep up with code reviews

Joe Gordon joe.gordon0 at gmail.com
Fri May 31 04:48:27 UTC 2013


On Thu, May 30, 2013 at 7:16 PM, Russell Bryant <rbryant at redhat.com> wrote:

> On 05/30/2013 05:42 PM, Joe Gordon wrote:
> > On a related note, if you search for "is:open CodeReview+2 Verified+1
> > -Verified-1 -CodeReview-2 is:watched" on review.openstack.org
> > <http://review.openstack.org> you can see what patches have no negative
> > feedback and at least one +2 already.
>
> Thanks for pointing out some additional query parameters I didn't know
> about.
>
> However, I would caution against using this particular query *too* much.
>  I think people often look for reviews with a +2 already, because
> they're looking for easy reviews.  I think that's counter to the
> criteria both reviewday and next-review are promoting (things that are
> high priority based on bug/blueprint status, or because they have been
> waiting the longest).
>

++, I should have put a caution, use sparingly, statement next to the
query.


>
> --
> Russell Bryant
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20130530/781e1512/attachment.html>


More information about the OpenStack-dev mailing list