Your teammate asks you to review his Pull Request. But what should you check to not miss anything? Here is my quick checklist for Reviewer.

Code review checks for Reviewer

During a code review please check that:

  1. There are no architectural/design issues
  2. There are no antipatterns
  3. There are no “code smells” characteristics
  4. The code can be easily maintained, changed and/or scaled
  5. The code adheres to proper naming convention of functions and variables
  6. There is no debug information and print/echo statements
  7. There isn’t any commented out code
  8. If some code needs to be commented out, there is a TODO mark with name, email and description why it’s commented out
  9. There is a new line at the end of files
  10. There is a Shebang if needed
  11. There are no unused imports, variables, functions
  12. If Submitter did refactoring, he cleaned all the code related to his refactoring
  13. The README.md file has been created or updated (if needed)
  14. All unnecessary files and folders are deleted or added to .gitignore

Conclusion

To make the process of reviewing Pull Requests more enjoyable and expected, you need a set of rules or recommendations. In this post you can find my thoughts on which rules and recommendation you can apply to make the process better.

What do you usually check during review? Please, share it with me!