Code reviews are great, they produce better code, and allow convergence in style
and approach, without the authoritarian feel of formal best practices
documentation. Avoiding formality is something I hold dear, as creative free
thinking individuals are who I want to be around, and who I want to be. The
problem with code reviews, is that creative free thinking individuals will
always have creative free thinking differences. This causes tension.
What I really want is a code review where I don't have to suppress the urge to
defend my less elegant choices. A code review where my lizard brain sense of
pride doesn't have a say.
The problem is that this defensiveness is natural, you poured your heart and
soul, 3 days of your life into that final change set. Having someone you respect
nitpick about that 10 line method somewhere in the 500 lines of change is hard.
We both know it's shit, but the idea that your masterpiece has a flaw, and a
flaw that may take another day of your life to fix is hard to swallow. This is
A computer saying the same thing does not tickle the same emotions.