Peer Code Review: Difference between revisions

Revision as of 20:53, 17 November 2010

formal inspection: author; reveiwer (who ndoes the explained based on the review, no author input till meeting); observer

tool assisted: file gathering; combined display: diff comments defects; automated metric collection; review enforcement,

phase-injection: developer error, design oversight, requirements mistake, QA error

@@ Line 3: / Line 3: @@
 ;formal inspection
 : author
-; reveiwer (who ndoes the explained based on the review, no author input till meeting)
+: reveiwer (who ndoes the explained based on the review, no author input till meeting)
-; observer
+: observer
-* over-the-shoulder
-* pair prog
+; over-the-shoulder
-* tool assisted: file gathering, combined display: diff comments defects, automated metric collection, review enforcement,
+: one walks through the code while the other watches.
-* email pass around
+; pair programming
+: like in XP/agile
+; tool assisted
+: file gathering
+: combined display: diff comments defects
+: automated metric collection
+: review enforcement,
+; email pass-around
 == Code Errors ==
@@ Line 14: / Line 24: @@
 ;severity
 : major, minor
 ;type
 : algorithm, documentation, data-usage, error-handling, input, output
 ; phase-injection
 : developer error, design oversight, requirements mistake, QA error
-== biblio ==
+== Bibliography ==
-best kept xsecrets of peer code review
+* Best Kept Secrets of Peer Code Review - Jason Cohen 2006
-* code inspection - michael fagan ibm 1974 (29pgs)
+* Code Inspection - Michael Fagan IBM 1974 (29 pages)
 [[Category:Computers]]