“We don’t do code reviews
“We don’t do code reviews. We trust our developers.”
<insert epic eye roll here>
No one should be blindly trusted.
No one is beyond making a mistake.
No one is “above” needing feedback.
Code should be reviewed.
I don’t want to walk across a bridge built by a bunch of different people who were “trusted” enough that their final work wasn’t inspected.
We’re all fallible. A second set of eyes improves quality.
“We pair program, so we don’t need code reviews”
I disagree. A code review by others remains valuable.
People who didn’t write the code have a fresh perspective. So they often find bugs, bad tests, and unforeseen impacts. They can point out parts that are unclear to outsiders.
Oh, and there are other code review benefits:
1. Gives the team a chance to see the changes in isolation before they merge.
2. Allows the team to discuss and address code conflicts before they occur.