Talk in English - US at Cascadia PHP
Track Name:
Crater Lake
View Slides: https://stevegrunwell.com/slides/code-review
Short URL: https://joind.in/talk/bcbbb
(QR-Code (opens in new window))
On the surface, the idea of code review is a no-brainer: why wouldn’t we want a second set of eyes on our code, especially before deploying to production?
As we peel back the layers, however, we find that the topic of code review is much more nuanced. How detailed should the review be? Who is qualified to perform the review (hint: it’s not just senior developers)? Can we afford to take another developer away from their project to review this one? What steps can we take to ensure reviews are constructive, rather than demoralizing?
Attendees will gain deeper insight into some of the arguments for and against systemic, peer code review, as well as pick up some useful tools to make code review a natural part of their teams’ workflow.
Comments
Comments are closed.
This talk contained a lot of great advice both for existing code reviewers and organizations that are considering implementing code review. I appreciated the examples of highlighting successes and doling out the warm fuzzies. The discussion of bus factor (or lottery factor) was also appreciated, because it helped to establish an important but overlooked benefit of code review: cross-organization education and knowledge sharing.