Four NOs of a Serious Code Reviewer by Yegor Bugayenko

Speaker Yegor Bugayenko
Title
Four NOs of a Serious Code Reviewer
Type
  • talk
Duration
  • 60
Track
  • engineering
Description

Code reviews (a.k.a. peer reviews) must be a mandatory practice for every serious software development team. Some do pre-merge code reviews, protecting their master/development branch from accidental mistakes. Others do post-merge regular reviews to discover bugs and inconsistencies after they are introduced by their authors. Some even do both, reviewing before merges and regularly after. However, it's not so simple to do them right. I would even say it's very easy and comfortable to do them wrong. Most code reviews and reviewers I've seen make similar mistakes. That's why I decided to summarize the four basic principles of a good reviewer as I see them.

Agenda

I've never presented this talk anywhere, but wrote a blog post about this subject: http://www.yegor256.com/2015/02/09/serious-code-reviewer.html

Constraints and Class Arrangement
Session History
Co-presenter
Last Updated 24 Nov 08:27