Your SlideShare is downloading. ×
JDD Effective Code Review In Agile Teams
Upcoming SlideShare
Loading in...5

Thanks for flagging this SlideShare!

Oops! An error has occurred.

Saving this for later? Get the SlideShare app to save on your phone or tablet. Read anywhere, anytime – even offline.
Text the download link to your phone
Standard text messaging rates apply

JDD Effective Code Review In Agile Teams


Published on

Presentation about effective code review for agile developers. …

Presentation about effective code review for agile developers.
Slides were used during sessions at Agile2009 in Chicago and JDD09 in Krakow.

Published in: Technology

  • Be the first to comment

No Downloads
Total Views
On Slideshare
From Embeds
Number of Embeds
Embeds 0
No embeds

Report content
Flagged as inappropriate Flag as inappropriate
Flag as inappropriate

Select your reason for flagging this presentation as inappropriate.

No notes for slide


  • 1. Effective Code Review for Agile Java Developers Wojciech Seliga & Sławomir Ginter Tag cloud generated with Licensed under a Creative Commons Attribution-NonCommercial-ShareAlike 2.5 License.
  • 2. An Evil Plan Picture courtesy of Helico CC BY 2.0
  • 3. What code review has to do with Agile
  • 4. Picture courtesy of Cat and Girl CC A-NC-SA 2.5
  • 5. style/conventions design reusable components APIs do's and don'ts Picture courtesy of mil8 / CC BY 2.0
  • 6. Mentoring junior developers Non-intrusive Asynchronous Less frustration / interruption for senior devs Picture courtesy of eddidit / CC BY 2.0
  • 7. Sharing good engineering practices Picture courtesy of Hey Paul / CC BY 2.0
  • 8. Picture courtesy of Jordan Miller / CC 2.5
  • 9. Photo Courtesy of U.S. Army
  • 10. Picture courtesy of tinyfroglet / CC 3.0
  • 11. Preparation is difficult selecting the code organizing reviewers booking conference room scheduling printing ... Picture courtesy of jfdervin / CC BY-SA 2.0
  • 12. synchronized meetings de-focusing
  • 13. Picture courtesy of gadl / CC BY-SA 2.0
  • 14. Risk of animosities
  • 15. No concrete measureable results Picture courtesy of aussiegall / CC BY 2.0
  • 16. Successful code review Lightweight - simple & flexible process Asynchronous Continuous Efficient tool support Diff-oriented whenever applicable Transparent and persistent
  • 17. Time for a longer demo +
  • 18. Agile code review misconceptions a fanatic bug-hunt false confidence about no bugs left tracking results of every single comment expecting hard metrics Picture courtesy of Juria Yoshikawa / CC BY-SA 2.0
  • 19. The greatest initial Enemies Rigid Process Metrics Micro-management But teams evolve... Picture courtesy of tms. / CC BY-NC-ND 2.0
  • 20. Some rules of agile code review everyone can join review & comment everyone can modify the scope of the review everyone can invite other people everything is public across the company it's about learning, it's not about blaming Picture courtesy of PantoDX / CC BY 2.0
  • 21. Unexpected advantages Facilitation of distributed teams Collaboration on low level design Easier to introduce than pair-programming Time-zone difference may help you Knowledge Base Picture courtesy of david.nathan.cox / CC BY 2.0
  • 22. Code Review vs. Pair Programming verification creation later sharing now knowledge ”repression” & ”prevention” asynchronous responsibility synchronous distributed improved quality co-located lower barrier higher barrier collaboration extensive intensive permanent volatile
  • 23. Picture courtesy of Kevin Dooley / CC BY 2.0
  • 24. About us Tag clouds generated with
  • 25. Q&A Picture courtesy of Mykl Roventine / CC BY 2.0
  • 26. Thank you