diff options
author | Chris Lattner <sabre@nondot.org> | 2007-12-03 19:00:47 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2007-12-03 19:00:47 +0000 |
commit | 85d1d0b9eaee401681ef13b24c027342ce44554a (patch) | |
tree | 6e811971b7db5c8a795afcfc4f8e95198c256a43 | |
parent | 9c945ce2229dd278974b4b7012c94421eef920fc (diff) | |
download | external_llvm-85d1d0b9eaee401681ef13b24c027342ce44554a.zip external_llvm-85d1d0b9eaee401681ef13b24c027342ce44554a.tar.gz external_llvm-85d1d0b9eaee401681ef13b24c027342ce44554a.tar.bz2 |
Describe the notion of 'owners' of the code.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@44537 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | docs/DeveloperPolicy.html | 45 |
1 files changed, 45 insertions, 0 deletions
diff --git a/docs/DeveloperPolicy.html b/docs/DeveloperPolicy.html index 647e2b8..883509b 100644 --- a/docs/DeveloperPolicy.html +++ b/docs/DeveloperPolicy.html @@ -15,6 +15,7 @@ <li><a href="#informed">Stay Informed</a></li> <li><a href="#patches">Making a Patch</a></li> <li><a href="#reviews">Code Reviews</a></li> + <li><a href="#owners">Code Owners</a></li> <li><a href="#testcases">Test Cases</a></li> <li><a href="#quality">Quality</a></li> <li><a href="#commitaccess">Obtaining Commit Access</a></li> @@ -146,6 +147,50 @@ </div> <!-- _______________________________________________________________________ --> +<div class="doc_subsection"> <a name="owners">Code Owners</a></div> +<div class="doc_text"> + + <p>The LLVM Project relies on two features of its process to maintain rapid + development in addition to the high quality of its source base: the + combination of code review plus post-commit review for trusted maintainers. + Having both is a great way for the project to take advantage of the fact + that most people do the right thing most of the time, and only commit + patches without pre-commit review when they are confident they are + right.</p> + + <p>The trick to this is that the project has to guarantee that all patches + that are committed are reviewed after they go in: you don't want everyone + to assume someone else will review it, allowing the patch to go unreviewed. + To solve this problem, we have a notion of an 'owner' for a piece of the + code. The sole responsibility of a code owner is to ensure that a commit + to their area of the code is appropriately reviewed, either by themself or + by someone else. The current code owners are:</p> + + <ol> + <li><b>Anton Korobeynikov</b>: Exception handling, debug information, and + Windows codegen.</li> + <li><b>Duncan Sands</b>: llvm-gcc 4.2.</li> + <li><b>Evan Cheng</b>: Code generator and all targets.</li> + <li><b>Chris Lattner</b>: Everything else.</li> + </ol> + + <p>Note that code ownership is completely different than reviewers: anyone can + review a piece of code, and we welcome code review from anyone who is + interested. Code owners are the "last line of defense" to guarantee that + all patches that are committed are actually reviewed.</p> + + <p>Being a code owner is a somewhat unglamorous position, but it is incredibly + important for the ongoing success of the project. Because people get busy, + interests change, and unexpected things happen, code ownership is purely + opt-in, and anyone can choose to resign their "title" at any time. For now, + we do not have an official policy on how one gets elected to be a code + owner. + </p> + +</div> + + +<!-- _______________________________________________________________________ --> <div class="doc_subsection"> <a name="testcases">Test Cases</a></div> <div class="doc_text"> <p>Developers are required to create test cases for any bugs fixed and any new |