Code reviews are an amazing opportunity for teams to help one another grow and maintain a high bar of quality for their code bases overall. Unfortunately this opportunity is often squandered and both giving and receiving these reviews can easily become one of the most frustrating parts of our jobs. Come learn about tactics for improving the process on your team.
30. Atlassian Crucible Atlassian's code review tool
Google Gerrit Open source git code review tool
Github Git hosting and pioneer of the "Pull Request"
Facebook Phabricator Open source git/mercurial/svn review tool
Review Board Open source review tool that is SCM/platform neutral
JetBrains Upsource git/mercurial/perforce/svn review tool
Visual Studio Team Services collaborative environment with git
Code review tools
35. Importance of human code review
“Code Review is the single greatest way of
noticing and killing bugs, increasing overall
understanding, fixing design problems, and
learning from one another.”
-Graydon Hoare
50. ?
“The zoombinator is
throwing a zoom
exception and I thought
this would fix the issue. I
can take another look if
you think that would be
better.”
60. Absence of trust
Fear of conflict
Lack of commitment
Avoidance of accountability
Inattention to results
61. Absence of Trust -> Build Trust
Fear of Conflict -> Be Honest and Open
Lack of Commitment -> Work as a Team
Avoidance of Accountability -> Be Accountable
Inattention to Results -> Tune in
63. Be empathetic (Project Aristotle)
People were skilled at reading emotions based on nonverbal cues.
If a team member appeared uneasy with a decision, it was likely
noticed and discussed. If someone appeared down, others
showed concern and support. Those conversations are not always
easy, but they’re important. They allow us to be authentic and
engaged.
72. Speak like you like each other
Make the other person feel safe
The focus is NOT “being right”
Say “What was the reasoning here...” instead of “why”
99. “Did you consider that the
Zoombie class allows us
to filter bad requests?”
100. ?
“The zoombinator is
throwing a zoom
exception and I thought
this would fix the issue. I
can take another look if
you think that would be
better.”
123. ● Icons taken from flaticon.com
● “The 5 Dysfunctions of a Team” by Patrick Lencioni
● “Code Complete” by Steve McConnell
● “Best Kept Secrets of Peer Code Review” by Jason Cohen
● The New York Times
● Medium
Sources