Code review “on a napkin” — could it be useful?
- by gaRex
Preconditions
Team uses DVCS
IDE supports comments parsing (like TODO and etc.)
Tools like CodeCollaborator are expensive for budget
Tools like gerrit are too complex for install or not usable
Workflow
Author publishes somewhere on central repo feature branch
Reviewer fetch it and start review
In case of some question/issue reviewer create comment with special label, like "BLA". Such label MUST not be in production code -- only on review stage:
$somevar = 123;
// BLA Why do echo this here?
echo $somevar;
When reviewer finish post comments -- it just commits with stupid message "comments" and pushes back
Author pulls feature branch back and answer comments in similar way or improve code and push it back
When "BLA" comments have gone we can think, that review has successfully finished.
Author interactively rebases feature branch, stashes it to remove those "comment" commits and now is ready to merge feature to develop or make any action that usualy could be after successful internal review
IDE support
I know, that custom comment tags are possible in eclipse & netbeans. Sure it also should be in blablaStorm family.
So my specific questions are
Do you think this methodology is viable?
Do you know something similar?
What can be improved in it?
ps: migrated from http://stackoverflow.com/questions/12692695/code-review-on-a-napkin-could-it-be-useful
