Loading…
This event has ended. View the official site or create your own event → Check it out
This event has ended. Create your own
View analytic
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 and see who's attending!

Limited Capacity full
Adding this to your schedule will put you on the waitlist.

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.

Speakers
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 integration environment. He also headed up the move to git and gerrit for department-wide pre-commit code reviews. He has been a part of the Philly tech scene for the... Read More →



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

Attendees (101)


Waitlist (13)