[webkit-dev] Asking for review by pinging bugs---another approach

[Gyuyoung Kim]

In new port case, for example EFL port, we unfortunately don't have special
reviewer for our port yet.
So, we have requested to review on IRC after finishing our informal review.
If we only
ask some reviewers to review, the reviewers will have too many review
burden. We have
found proper reviewers on IRC in order to avoid this. In this case, is
there better way to
find proper reviewer for EFL port patch ?

- gyuyoung


On Fri, Jan 6, 2012 at 3:40 AM, Adam Barth <abarth at webkit.org> wrote:

> Sure.  Feel free to add that feature.  Obviously we don't want to force
> people to use it, but it might become popular.
>
> Adam
>  On Jan 5, 2012 3:12 AM, "Hajime Morrita" <morrita at chromium.org> wrote:
>
>> Does it make sense for webkit-patch post/upload to support "--ping" to
>> acknowledge a set of reviewers using a specific form of comment? As a
>> reviewer, some kind of specific pattern would be helpful to filter
>> such ping mails which contain, for example, "ping:morrita@".
>>
>> Maybe this can be taken as a step backward to the "by name" review
>> request which we had declined. But at least for me, such kind of
>> filterable mail would be gret tool to pick bugs for review.  Even
>> though this won't help some high-traffic reviewers, it won't hurt at
>> least. I've been overwhelmed by the stream of Bugzilla mail coming...
>>
>> What do you think?
>>
>> --
>> morrita
>>
>>
>> On Thu, Jan 5, 2012 at 5:03 PM, Andreas Kling <kling at webkit.org> wrote:
>> > On Thu, Jan 5, 2012 at 7:49 AM, Adam Barth <abarth at webkit.org> wrote:
>> >>
>> >> Not to pick on anyone in particular, but when reading bugmail I
>> >> occasionally see messages like "pinging for review."  I review a lot
>> >> of patches, but I don't find these messages particularly helpful
>> >> because I don't know whether I'm supposed to review the patch.
>> >>
>> >> Another approach that might work better for you is to address your
>> >> comment at someone in particular.  For example, if the message says
>> >> "Adam, can you please review this patch?", then there's a pretty good
>> >> chance I'll click through and try to answer your question.
>> >>
>> >> If you're unsure who to ask for review, one approach is to look at the
>> >> svn log for the files you're changing and see who has written/reviewed
>> >> patches for those files recently.  You can also ask folks who've been
>> >> around the project for a while to suggest someone.
>> >
>> >
>> > True that.
>> >
>> > You can also find reviewers for a particular area here on the WebKit
>> Team
>> > wiki page: http://trac.webkit.org/wiki/WebKit%20Team (and I encourage
>> people
>> > to keep their entry up-to-date.)
>> >
>> > -Kling
>> >
>> > _______________________________________________
>> > webkit-dev mailing list
>> > webkit-dev at lists.webkit.org
>> > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>> >
>>
>
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20120109/695e1496/attachment.html>