[webkit-dev] Review Queue

[Eric Seidel]

Ok, I agree that a sectioned queue would help.  There is no need to
see the Gtk patches in the list:
https://bugs.webkit.org/buglist.cgi?field0-0-0=flagtypes.name&type0-0-0=equals&value0-0-0=review%3F
(which is back up to 40, btw!)

But we should be able to keep "core" (nearly) empty if it were kept separate.

-eric

On Fri, May 22, 2009 at 11:22 PM, Eric Seidel<eric at webkit.org> wrote:
> We're down to 30 bugs:
> https://bugs.webkit.org/buglist.cgi?field0-0-0=flagtypes.name&type0-0-0=equals&value0-0-0=review%3F
>
> With 36 patches total:
> curl -s "https://bugs.webkit.org/request.cgi" | grep PDT | wc -l
>
> 15 of those are Gtk-only bugs.  Need some help from the Gtk reviewers
> on this one.
>
> Thanks again to those who have pitched in so far.
>
> -eric
>
> On Sat, May 23, 2009 at 12:20 AM, Eric Seidel <eric at webkit.org> wrote:
>> Reviewed more before bed.  We're down to:
>>
>> https://bugs.webkit.org/buglist.cgi?field0-0-0=flagtypes.name&type0-0-0=equals&value0-0-0=review%3F
>> 50
>>
>> and
>>
>> curl -s "https://bugs.webkit.org/request.cgi" | grep PDT | wc -l
>> 61
>>
>> respectively.
>>
>> Still awful, but much much better than yesterday.
>>
>> -eric
>>
>> On Fri, May 22, 2009 at 7:40 PM, Maciej Stachowiak <mjs at apple.com> wrote:
>>>
>>> On May 22, 2009, at 2:19 AM, Eric Seidel wrote:
>>>
>>>> Update:  We're down to 74 patches now.
>>>>
>>>> Thanks especially to Maciej for all his reviewing this evening:
>>>>
>>>> curl -s "https://bugs.webkit.org/request.cgi" | grep PDT | wc -l]
>>>> 74
>>>>
>>>> Still a long way to go.
>>>
>>> FWIW I prefer this query, which counts by bug and gives a count of 57:
>>>
>>> https://bugs.webkit.org/buglist.cgi?field0-0-0=flagtypes.name&type0-0-0=equals&value0-0-0=review%3F
>>>
>>> It looks like about half the remaining bugs are Qt or Gtk specific, and
>>> likely need someone who is expert on those ports to look at them. I don't
>>> feel comfortable reviewing things that affect Qt or Gtk APIs especially.
>>>
>>> - Maciej
>>>
>>>>
>>>>
>>>> -eric
>>>>
>>>> On Fri, May 22, 2009 at 12:27 PM, Eric Seidel <eric at webkit.org> wrote:
>>>>>
>>>>> Our review process seems to be failing.  As a reviewer, let me extend
>>>>> my apologies to the WebKit community as I am part of this failure.
>>>>>
>>>>> We have over 100 patches in the review queue at the moment:
>>>>> http://webkit.org/pending-review
>>>>>
>>>>> I've started going through the list and reviewing what patches I can.
>>>>> I'm also marking r- all patches I can't review which have had no
>>>>> comments in the last 2 weeks.
>>>>>
>>>>> -eric
>>>>>
>>>> _______________________________________________
>>>> webkit-dev mailing list
>>>> webkit-dev at lists.webkit.org
>>>> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>>>
>>>
>>
>