Your SlideShare is downloading. ×
15 Minute Code Review  Quickstartbrian d foy, brian.d.foy@gmail.com          The Perl Review
Makebetter programmers         not  better programs
Choose a reason• Inspect work• Mentor junior members• Sharpen senior members• Cross-pollinate ideas• Find bugs
Start small• Start with small goals• Make it a regular, expected event• Reach a goal? Extend it.• Refine and document the ...
Set boundaries• Be nice• Limit the scope and depth of code• Limit the time for each item• Don’t ambush• Don’t make correct...
Review code that...• compiles• passes its tests • is covered by the tests • is profiled• is under development
Stay on target• Make a checklist• Show up prepared• Facilitate discussion but defer arguments• Quash digressions (send to ...
Starter checklist• Is documented• Everyone understands • what it does • how it does it • why it does it that way• Follows ...
Steal from others• Effective Code Reviews Without the Pain    http://goo.gl/8bWO•   Five Whys    http://5whys.com/
Upcoming SlideShare
Loading in...5
×

Code Review Quickstart

2,003

Published on

Start your team reviewing your programming work. Become a better programmer!

Published in: Technology
0 Comments
4 Likes
Statistics
Notes
  • Be the first to comment

No Downloads
Views
Total Views
2,003
On Slideshare
0
From Embeds
0
Number of Embeds
1
Actions
Shares
0
Downloads
1
Comments
0
Likes
4
Embeds 0
No embeds

No notes for slide

Transcript of "Code Review Quickstart"

  1. 1. 15 Minute Code Review Quickstartbrian d foy, brian.d.foy@gmail.com The Perl Review
  2. 2. Makebetter programmers not better programs
  3. 3. Choose a reason• Inspect work• Mentor junior members• Sharpen senior members• Cross-pollinate ideas• Find bugs
  4. 4. Start small• Start with small goals• Make it a regular, expected event• Reach a goal? Extend it.• Refine and document the process
  5. 5. Set boundaries• Be nice• Limit the scope and depth of code• Limit the time for each item• Don’t ambush• Don’t make corrections• End on time
  6. 6. Review code that...• compiles• passes its tests • is covered by the tests • is profiled• is under development
  7. 7. Stay on target• Make a checklist• Show up prepared• Facilitate discussion but defer arguments• Quash digressions (send to hallway track)• Note problems and move on
  8. 8. Starter checklist• Is documented• Everyone understands • what it does • how it does it • why it does it that way• Follows local style
  9. 9. Steal from others• Effective Code Reviews Without the Pain http://goo.gl/8bWO• Five Whys http://5whys.com/

×