Probably not you… or me, or anyone. BUT – in our current state of code, this is still the BEST tool to control code quality, get other team members to be familiar with your code, and avoid stupid mistakes.
Still – it’s a mystery of how to do it RIGHT. Everyone’s got their own standards for CR and arguments aren’t too scarce to be found.
What can solve this? A firm, clear policy everyone could follow. Maybe something like this for a start: