WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
31313
Make 80 patches a formal requirement for reviewership
https://bugs.webkit.org/show_bug.cgi?id=31313
Summary
Make 80 patches a formal requirement for reviewership
Eric Seidel (no email)
Reported
2009-11-10 14:17:20 PST
Created
attachment 42895
[details]
Patch After discussion on webkit-reviewers, make 80 patches a formal requirement for reviewers.
Attachments
Patch
(1.13 KB, patch)
2009-11-10 14:17 PST
,
Eric Seidel (no email)
no flags
Details
Formatted Diff
Diff
Patch
(1.76 KB, patch)
2009-11-10 14:19 PST
,
Eric Seidel (no email)
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1
2009-11-10 14:19:11 PST
Created
attachment 42896
[details]
Patch
Darin Adler
Comment 2
2009-11-10 20:50:21 PST
I have mixed feelings about this, and would like to at least hear from Maciej before we make this change.
Eric Seidel (no email)
Comment 3
2009-11-13 13:47:34 PST
CCing Maciej per Darin's request.
David Levin
Comment 4
2009-11-17 15:22:01 PST
Comment on
attachment 42896
[details]
Patch No one objected on webkit-reviewers. Also, this seems reasonable and in alignment with the curretly accepted practice.
WebKit Commit Bot
Comment 5
2009-11-18 16:18:13 PST
Comment on
attachment 42896
[details]
Patch Clearing flags on attachment: 42896 Committed
r51152
: <
http://trac.webkit.org/changeset/51152
>
WebKit Commit Bot
Comment 6
2009-11-18 16:18:23 PST
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug