[webkit-dev] webkit.org/pending-review

[Maciej Stachowiak]

On Jan 14, 2010, at 12:25 AM, Eric Seidel wrote:

> Oliver Hunt mentioned this evening that he found the
> webkit.org/pending-review page confusing because it lists both
> commit-queue? patches and review? patches.  I agreed.
>
> I filed https://bugs.webkit.org/show_bug.cgi?id=33656 to track
> updating /pending-review to more modern review URL and made some
> possible suggestions as to better URLs.  Maciej is the man with the
> real URL magic though. :)
>
> I'm not sure how /pending-review is managed, it does not seem to be
> stored in WebKitSite/, perhaps someone on this list would know how to
> change it?  Or would maybe folks have their own suggestions as to what
> URL we should use for /pending-review (if so please update the bug).
>
> /pending-review is used by http://nightly.webkit.org/start and may be
> linked to from other places in the site, I don't know.

The page that pending-review points to actually has some nice  
properties in the way it lays things out, but in other ways its harder  
to read. I'd really like it if we could make a review queue page  
that's combines their best features and is nicer than either. This may  
be hard. The queries I personally like to use these days are:

http://tinyurl.com/y98qfjs (general patches, no bracket)
http://tinyurl.com/yjr2xru (patches needing port-specific review,  
[bracket] prefix)

Their main downside is that they do not sort by date of review request.

Regards,
Maciej

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20100114/a6e6c56e/attachment.html>