Changes between Version 4 and Version 5 of PatchReviewProcess


Ignore:
Timestamp:
2010-02-16T02:57:46Z (14 years ago)
Author:
davidsarah
Comment:

assign to someone with repository write access

Legend:

Unmodified
Added
Removed
Modified
  • PatchReviewProcess

    v4 v5  
    2424  a. If you can't understand the patch after spending some time on it, then say so in a comment on the ticket!  This might be taken as a reason to add documentation or comments or to refactor the code.  On the other hand, it might just be that you don't have enough context to understand the code.  That's okay too.
    2525  b. If you find errors or omissions in the docs, tests, code or comments then write that down in the ticket, remove the "review" keyword from the keywords, and assign the ticket to someone other than yourself.  (That would be the original author of the patch, or someone who seems likely to fix the patch, or if you can't think of anyone better then assign it to me.)
    26   c. If you understand the patch and find no errors or omissions then remove the keyword "review", add the keyword "reviewed" and assign it to me. I'll commit it to trunk.
     26  c. If you understand the patch and find no errors or omissions then remove the keyword "review", add the keyword "reviewed" and assign it to someone with repository write access (currently 'zooko', 'warner' and 'davidsarah'). We'll commit it to trunk.
    2727  d. Feel good about yourself.  Thank you for helping with our little project attempting to improve the world!