Abstract/Description

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.

Additional Resources

About the Speaker(s)

Trevor Lalish-Menagh is a lead software engineer at thePlatform where he leads a small team of engineers and testers to create the backend systems that run a large part of the online video offerings available on the Internet today. He is passionate about adapting processes to fit the team and achieving high code quality. He has spoken nationally on topics of Agile, kanban, testing, and code quality.