For a good PR review you should consider the following topics It's proposal should be clear Code quality Readability Unexpected exception Security Issues Unexpected recursion Test readability Test coverage