· daily

Code review practices - Review in logical sets

Instead of looking at code sequentially, club it together into logicaly changesets

  • Look at architecture and design
  • Review from business logic viewpoint
  • Look for performance issues
  • Security problems
  • Code formatting, naming conventions styles (ideally first pass can be checked with tools automatically)
  • Examine testing pieces
  • Look for documentation

Hopefully this will catch more issues and provide effective feedback