Bug 47597

Summary: Proposal: Require cross-organizational seconds for reviewer nominations
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: New BugsAssignee: Eric Seidel (no email) <eric>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, darin, hamaji, kenneth, kling, mjs, tonikitoo
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch darin: review+

Description Eric Seidel (no email) 2010-10-13 09:49:16 PDT
Proposal: Require cross-organizational seconds for reviewer nominations
Comment 1 Eric Seidel (no email) 2010-10-13 09:55:38 PDT
Created attachment 70619 [details]
Patch
Comment 2 Eric Seidel (no email) 2010-10-13 09:58:21 PDT
Created attachment 70620 [details]
Patch
Comment 3 Kenneth Rohde Christiansen 2010-12-24 02:44:19 PST
Comment on attachment 70620 [details]
Patch

I do not know who could review this, thought I support it. Did we agree on this on the mailing list?
Comment 4 Eric Seidel (no email) 2010-12-24 08:33:14 PST
I'm not as passionate about this patch as a once was. But I still think it's a good idea.
Comment 5 Antonio Gomes 2010-12-24 09:00:30 PST
I do not think this patch should be up for review until there is a 100% agreement on the mailing list.

ps: I agree with it. Well, sorta ... sometimes it is unneeded, you know...
Comment 6 Darin Adler 2010-12-24 09:50:43 PST
Comment on attachment 70620 [details]
Patch

This should be discussed on the reviewer mailing list. I don’t think the patch review process is sufficient for changing the rules about becoming a reviewer.
Comment 7 Adam Barth 2011-01-12 16:39:06 PST
Comment on attachment 70620 [details]
Patch

There's been some discussion of this change on the webkit-reviewers list.  I'm not sure this patch reflects the current consensus.  In any case, we should probably sort out what policy we want first.
Comment 8 David Levin 2011-01-13 11:13:00 PST
Created attachment 78830 [details]
Patch
Comment 9 Eric Seidel (no email) 2011-01-13 12:27:32 PST
Thanks dave!
Comment 10 David Levin 2011-01-13 15:25:24 PST
Created attachment 78863 [details]
Patch
Comment 11 Darin Adler 2011-01-21 11:00:19 PST
Comment on attachment 78863 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=78863&action=review

OK. Lets do this. My grammatical tweak suggestions are not binding, of course. For one thing they haven’t even been discussed with the rest of the reviewers.

> Websites/webkit.org/coding/commit-review-policy.html:102
> +community as a whole, and not just within their own place of employment or

This comma should be removed.

> Websites/webkit.org/coding/commit-review-policy.html:105
> +with the nominee (other than WebKit itself).</p>

This would be better with a comma rather than parentheses. I also think "the WebKit project itself" would be clearer than "WebKit itself".
Comment 12 Eric Seidel (no email) 2011-01-21 12:35:49 PST
http://trac.webkit.org/changeset/76367