After having tried out various review methods like direct review, reviews over email, pair programming etc, I have found that a combination of tool-assisted code review and direct review (on an as needed basis) works out to be the best in terms of efforts spent and quality of the output.
SmartNBear.com:Pros and Cons of Four Kinds of Code mentions the pros and cons of the various types of reviews.
My first experience with Crucible by Atlassian was a bit unnerving. However there were many obvious advantages that I quickly noticed:
- Making a comment right where it make sense. i.e. near a particular line of code. This is a killer feature.
- Enforcement of workflow
- Close integration with Perforce (since we use Perforce) Continue Reading