Code Review Guide » History » Revision 2
Revision 1 (Amber Herold, 03/04/2010 03:38 PM) → Revision 2/13 (Amber Herold, 03/04/2010 04:30 PM)
h1. Code Review Guide
[[Purpose of Code Reviews|What's the point of code reviews]]?
h2. The Process
# Developer writes code
# Developer checks code into Subversion
# Developer updates Redmine issue with:
## the svn revision number
## a description of the changes
## a reference to test cases or a description of how to test the changes
## set the Status to In Code Review
## assign the issue to another person to perform a code review
# Code reviewer receives an email that they have code to review
# Code reviewer inspects the changes to the code using the review guide.
## Revisions that involve complicated logic or widespread changes are better done in person. In this case the reviewer can ask the developer to do a walk through.
# If the reviewer finds a problem, the Redmine issue is updated with:
## a description of the problem
## the Assigned to field is set back to the developer and the process starts over.
# If no problems are found, the REdmine issue is updated with:
## The Status is set to In Test
## The Assigned to field is set to someone who can readily test it.