Do you wonder how to teach your team to make code reviews that are not a waste of time? My suggestions so far:
1. If you focus on code formatting you’re not doing it right. When you have tools, hooked (pre-commit hook) which checks that for you – it’s better.
2. Use your brainpower for something simple tools can not do – try to understand what you read. As long as code compiles it’s ok for machine. Reviews are meant to make your team write code for (other) people.
3. Reviewer HAVE TO spot issues. She/he can have solutions, but when there’s none – a discussion should be initialized.
5. Some action points after review would be “make refactor of this… some day”. This is at your team discretion if it’s possible to apply suggestions after review – sometimes it just can not be done.
6. Sometimes you have to back your opinion. I remember explaining many times why commented code is worse than deleted code. Now others slap me with this when I leave some code for later: