[webkit-dev] review queue crazy idea

Ryosuke Niwa rniwa at webkit.org
Wed Jul 21 22:17:54 PDT 2010


On Wed, Jul 21, 2010 at 2:40 PM, Ojan Vafai <ojan at chromium.org> wrote:

> There are currently 38 (of 171 total) patches in the review queue where
> the bugs have not been modified in over 1 month old. I propose we have a bot
> that educates people about writing easy to review patches and auto-rejects
> any patches in bugs that haven't been touched in over a month.


I'd love to see a bot that educates people but I'm not sure if we should
auto-rejects patches that are simply old. Maybe we need a little bit more
of graduality like pinging the author of the patch first, and then make it
obsolete if nobody responds etc...

This is just an initial proposal. I'm not wed to any of the details of how
> this would work. I do think that auto-rejecting old patches is valuable to
> the project as a whole. Having the review queue be so large makes
> it daunting for any reviewer to try and tackle it.
>

I'd agree that having a really large review queue isn't ideal.  Maybe we can
customize the review queue so that it only shows patches of your expertise.

Best,
Ryosuke Niwa
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20100721/9068dc73/attachment.html>


More information about the webkit-dev mailing list