code reviews don’t have to suck: best practices to streamline code reviews in eclipse (a tasktop...

17
Code Reviews Don’t Have to SUCK Roy Paterson Lead Developer David Shepherd Senior Developer

Upload: smartbear

Post on 19-Nov-2014

1.601 views

Category:

Self Improvement


0 download

DESCRIPTION

Visit the link below to download a free trial of CodeCollaborator http://smartbear.com/products/software-development/code-review/free-codecollaborator-trial Link to Webinar Video - http://www.slideshare.net/SmartBear_Software/code-reviews-dont-have-to-suck-best-practices-to-streamline-code-reviews-in-eclipse-a-tasktop-and-codecollaborator-webinar-7860807 Dr. David Shepherd, from Tasktop, and Roy Paterson, from SmartBear, demonstrate the latest in code review best practices, task focused development, and how to make code reviews not suck. In addition to the discussing the challenges associated with traditional code review techniques. The video version contains an interactive task driven code review.

TRANSCRIPT

Page 1: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Code Reviews Don’t Have to SUCK

Roy PatersonLead Developer

David ShepherdSenior Developer

Page 2: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Code Reviews Don’t Have to SUCK

Roy PatersonLead Developer

David ShepherdSenior Developer

Page 3: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Do Code Reviews Suck?

Page 4: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Yes: Meetings Suck

Intimidating and

Stressful

Page 5: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Yes: Meetings Suck

Bottleneck at

Author

Page 6: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Yes: Meetings Suck

Interruption from

Developer’s Day

Page 7: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Yes: Meetings Suck

Tool Disconnect Lowers Discussion Quality

Page 8: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Yes: Logistics Suck

Page 9: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Yes: Tool Support Sucks

Page 10: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Yes: Tool Support Sucks

Page 11: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Yes: Tool Support Sucks

Page 12: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

CodeCollaborator + TasktopEliminating the Suck from Code Reviews

No Meetings Feedback occurs asynchronously,

online

Automated Logistics are automated and

seamless

Tool Support Developers create + review code

in IDE

Page 13: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

CodeCollaborator + TasktopToday’s Demo Structure

Create Author creates review

Review Reviewer adds comments,

defects

Complete After fixing feedback, commits code

Rework Author fixes code, uploads fixes

Verify Reviewer verifies code fixes

Page 14: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Demo

Page 15: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

CodeCollaborator + TasktopEliminating the Suck from Code Reviews

No Meetings Feedback occurs asynchronously,

online

Automated Logistics are automated and

seamless

Tool Support Developers create + review code

in IDE

Page 16: Code Reviews Don’t Have to Suck: Best practices to streamline code reviews in Eclipse (A Tasktop and CodeCollaborator Webinar)

Q&ARoy PatersonLead [email protected]

David ShepherdSenior Developer@davidcshepherd