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

2012-01-23 Thread 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

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

2012-01-23 Thread 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

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

2012-01-23 Thread 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

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

2012-01-09 Thread Gyuyoung Kim
Title: Samsung Enterprise Portal mySingle 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

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

2012-01-05 Thread Adam Barth
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" wrote: > Does it make sense for webkit-patch post/upload to support "--ping" to > acknowledge a set of reviewers using a specific

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

2012-01-05 Thread Hajime Morrita
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

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

2012-01-05 Thread Andreas Kling
On Thu, Jan 5, 2012 at 7:49 AM, Adam Barth 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

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

2012-01-04 Thread Adam Barth
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 bett