Writing a blog post for new reviewers doesn't add value to the blog
[WebKit-https.git] / Websites / webkit.org / coding / commit-review-policy.html
1 <?php 
2     $title="Policy on Committers and Reviewers";
3     include("../header.inc"); 
4 ?>
5 <h2>WebKit Committers and Reviewer Policy</h2>
6
7 <p>The WebKit project has two kinds of special status beyond being a
8 contributor. WebKit Committers have direct read-write access to the
9 Subversion repository, enabling them to commit changes by themselves
10 or reviewed changes by others if authors ask committers to do so. WebKit Reviewers are permitted to review
11 patches and may grant or deny approval for committing. Details of the
12 review and commit process are available
13 on the <a href="http://webkit.org/coding/contributing.html">contribution page</a>.</p>
14
15 <p>New WebKit Committers and WebKit Reviewers will be selected by the
16 set of existing WebKit Reviewers. We will create a mailing list,
17 &lt;<a href="mailto:webkit-reviewers@lists.webkit.org">webkit-reviewers@lists.webkit.org</a>&gt;,
18 for this purpose.</p>
19
20 <p>An up to date list of current WebKit Committers and WebKit
21 Reviewers will be maintained at webkit.org.</p>
22
23 <h3>Choosing Committers and Reviewers</h3>
24
25 <p>A candidate for WebKit Committer or WebKit Reviewer should
26 initially be nominated by a reviewer on the reviewers mailing list, in
27 accordance with the criteria below. If the required reviewers (see below)
28 second the nomination, then it carries within 5 business days unless
29 someone objects. If an objection is raised, the reviewers should
30 discuss the matter and try to come to consensus; failing this, the
31 matter will be decided by majority vote of the reviewers.</p>
32
33 <p>Once someone is successfully nominated for WebKit
34 Committer status, Apple will take care of sending the committer
35 agreement and setting up a Subversion account once signed and
36 received.</p>
37
38 <p>Once someone is successfully nominated for WebKit Reviewer status,
39 the nominating Reviewer or another responsible party should inform the
40 candidate and ask for indication of acceptance from the potential new
41 reviewer. If the candidate accepts,
42 <a href="http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py">committers.py</a>
43 will be updated.</p>
44
45 <h3>Criteria for Committers</h3>
46
47 <p>A WebKit Committer should be a person we can trust to follow and
48 understand the project policies about checkins and other matters.</p>
49
50 <p>Normally a potential Committer would be nominated once they have
51 submitted around 10-20 good patches, shown good judgment and
52 understanding of project policies, and demonstrated good collaboration
53 skills. To be nominated and seconded, they will have to interact
54 with more than one project reviewer. If someone submits many patches
55 but does not show good judgment or effective collaboration, that
56 contributor might not be nominated right away. If someone submits fewer patches than
57 this but has experience working in another WebKit-based or
58 khtml-based engine and has a track record of good collaboration with
59 the WebKit project, they may be nominated sooner.</p>
60
61 <p>Significant contributors to testing, bug management, web site
62 content, documentation, project infrastructure and other non-code
63 areas may also be nominated, even without the normal threshold of
64 patches.</p>
65
66 <p>A person who will be working under the supervision of a WebKit
67 reviewer on WebKit-related projects can be nominated if the reviewer
68 is willing to vouch for them and supervise them to ensure they follow
69 project policies on checkins. Supervision does not necessarily imply a
70 manager/employee relationship, just that you work with the potential
71 committer closely enough to make sure they follow policy and work well
72 with others.</p>
73
74 <p>All committer nominations require the support of three reviewers.
75 One reviewer nominates, two others second the nomination.</p>
76
77 <h3>Criteria for Reviewers</h3>
78
79 <p>A WebKit Reviewer should be a person who has shown particularly
80 good judgment, understanding of project policies, collaboration
81 skills, and understanding of the code. Reviewers are expected to
82 ensure that patches they review follow project policies, and to do
83 their best to check for bugs or other problems with the patch. They
84 are also expected to show good judgment in whether or not they review
85 a patch at all, or defer to another reviewer.</p>
86
87 <p>A potential Reviewer may be nominated once they have
88 submitted a minimum of 80 good patches. They should also be in touch
89 with other reviewers and aware of who are the experts in various
90 areas.</p>
91
92 <p>A person who submits many patches but does not show good
93 collaboration skills, code understanding or understanding of project
94 policies may never be nominated. Making unofficial reviews before you
95 become a reviewer is encouraged. This is an excellent way to show your
96 skills. Note that you should not put r+ nor r- on patches in such
97 unofficial reviews.</p>
98
99 <p>For Reviewer status, there is no supervision exception.</p>
100
101 <p>All reviewer nominations require the support of four reviewers. One reviewer
102 nominates, three reviewers second. Reviewers should be known in the WebKit
103 community as a whole, and not just within their own place of employment or
104 just within a specific organized project. Therefore, at least one of the
105 four supporting reviewers must not share any company or project affiliations
106 with the nominee (other than WebKit itself).</p>
107
108 <h3>Suspension and Revocation of Committer or Reviewer Status</h3>
109
110 <p>WebKit Committer or WebKit Reviewer status can be revoked by 2/3
111 vote of the reviewers, not including the person under consideration
112 for revocation.</p>
113
114 <p>Someone actively damaging the repository or intentionally abusing
115 their review privilege may have it temporarily suspended on the
116 request of any two Reviewers. In such a case, the requesting Reviewers
117 should notify the webkit-reviewers list with a description of the
118 offense. At this point, Reviewer or Committer status will be
119 temporarily suspended for one week, pending outcome of the vote for
120 permanent revocation.</p>
121 <?php
122     include("../footer.inc");
123 ?>