Talk given during Coolblue Behind the Scenes on July 19th 2016, explaining what code review is, how to improve the quality of your reviews and what Coolblue learned while scaling this process up to 150 developers.
13. Definition
Code Review is the act of
consciously and systematically
convening with one’s fellow programmers
to check each other’s code for mistakes
Source: https://smartbear.com/learn/code-review/what-is-code-review/
14. Code Review is the act of
consciously and systematically
convening with one’s fellow programmers
to check each other’s code for mistakes
Definition
15. Definition
Code Review is the act of
consciously and systematically
convening with one’s fellow programmers
to check each other’s code for mistakes
16. Definition
Code Review is the act of
consciously and systematically
convening with one’s fellow programmers
to check each other’s code for mistakes
30. Who is reviewing?
?
? X
?
✓
✓
!!
X
✓
Indenting!
Whitespace!
Trailing comma!
Bad naming!
Low test coverage!
Debug code!
Insecure code!
Does not compile!
31. Who is reviewing?
?
✓ !!
X
✓
Indenting!
Whitespace!
Trailing comma!
Bad naming!
Low test coverage!
Debug code!
Insecure code!
Does not compile!
32. Answer me these questions three...
When
to review?
What
to review?
Who
is reviewing?
33. Asking for a review
- Start with the obvious
“The goal of my change is...”
- Limit size of change set
- Reserve time in advance
- Implement any feedback you receive (or explain why not)
- Keep your commits clean
- All tests green!