Skip to end of metadata
Go to start of metadata
  • 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?
Labels:
None
Enter labels to add to this page:
Please wait 
Looking for a label? Just start typing.