...
Test cases (including unit tests) have been written
Documentation and comments have been written
No new TODOs. If there are, a written comment with a clear explanation of why it cannot be resolved
Formatting follows the style guide and generally passes linting
Functionality works for all use cases
Your code should work to the best of your ability; no one wants to deal with fixing code that’s “good enough” or “works most of the time” when they’re trying to focus on their own code
...