edit

[Imported from Trac: page PatchReviewProcess, version 7]
zooko 2010-03-19 05:26:54 +00:00
parent 95adeb9f25
commit 4bf1fdd91c

@ -17,7 +17,7 @@ more specialized knowledge to review, but most don't.
# How to review patches # How to review patches
1. Go to <http://tahoe-lafs.org> . Click on "[View Tickets](http://tahoe-lafs.org/trac/tahoe/wiki/ViewTickets)". Click on "[review-needed](http://tahoe-lafs.org/trac/tahoe/query?order=priority&keywords=~review-needed)". 1. Go to <http://tahoe-lafs.org> . Click on "[View Tickets](http://tahoe-lafs.org/trac/tahoe/wiki/ViewTickets)". Click on "[review-needed](http://tahoe-lafs.org/trac/tahoe/query?order=priority&keywords=~review-needed)".
2. You can read everything without registering, but to add comments or change tickets you have to be logged in. Registering is quick and easy -- click the "Register" link at the top right of the page. 2. You can read everything without registering, but to add comments or change tickets you have to be logged in. Registering is easy -- click the "Register" link at the top right of the page.
3. Read tickets until you find one that you can review. 3. Read tickets until you find one that you can review.
4. (optional) Click "accept". This marks you as the person reviewing this patch. If you don't want to commit to this then you can skip this step. 4. (optional) Click "accept". This marks you as the person reviewing this patch. If you don't want to commit to this then you can skip this step.
5. Read the patch until you understand the docs, tests, code and comments in it. You can use the "Browse source" button at the top of the page to read the current versions of the files that the patch changes. 5. Read the patch until you understand the docs, tests, code and comments in it. You can use the "Browse source" button at the top of the page to read the current versions of the files that the patch changes.