Skip to content

Code review

  • Solution match the problem
  • Explain the change to the reviewer
  • Does the change solve the problem?
  • Is the change consistent with the existing architecture and design?
  • Is the code easy to understand?
  • Naming
  • Class, method size
  • Single Responsibility per class
  • Has it been tested?
  • Automated tests & test coverage
  • Do a quick manual test
  • Has the technical debt increased?
  • If, yes, where is it documented and who has approved it?