Actions
Code Review Guide » History » Revision 2
« Previous |
Revision 2/13
(diff)
| Next »
Amber Herold, 03/04/2010 04:30 PM
Code Review Guide¶
What's the point of code reviews?
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.
Updated by Amber Herold over 14 years ago · 2 revisions