Agile2014 has ended
Thursday, July 31 • 14:00 - 15:15
Code Review: Why it matters (Trevor Lalish-Menagh) FULL

Sign up or log in to save this to your schedule, view media, leave feedback and see who's attending!

Limited Capacity seats available

Today, so many projects, both open and closed source suffer from lack of quality. Many of us are aware of the importance of testing in our development processes, but code review is something that is often overlooked or misunderstood.

What does it mean to review code? Find bugs? Complain about tabs vs. spaces? Or something more? Code review is about quality, humility and sanity checks, and a good one takes some skill to do right. I will go over some tips I have found useful over the years doing code reviews.

I was responsible for moving a large part of Comcast over to doing pre-commit code reviews using git and gerrit. Team by team, I helped transition developers from a culture of not reviewing code to making code reviews and testing second nature. I have given this talk at local conferences (CPOSC) and user groups, and believe the benefits of good code reviews are under-represented in the developer community.

I cannot stress enough how much good code review practice has changed the way we work for the better, and would love the opportunity to share that knowledge with the world.

avatar for Trevor Lalish-Menagh

Trevor Lalish-Menagh

Principal Software Engineer, Comcast
Trevor Lalish-Menagh is a principal software engineer at Comcast Interactive Media where he spearheaded the front-end unit testing initiative, and co-created EnvJasmine (github.com/trevmex/EnvJasmine), a headless self-contained framework for testing front-end code in a continuous... Read More →

Thursday July 31, 2014 14:00 - 15:15 EDT
Osceola 4