- Rebased with current master
- There are tests for new feature/bugfix
- Tests pass
- Changes arm relevant only to the current feature/bugfix[URC]
- Does it have proper logging?
- Does it use network? what if there's a network error?
- Does it write to disk? what if the disk is full?
- Have you considered other edge cases, other than "happy paths"? What is the edge case?
[URC] If there are other small changes like simple refactorings, whitespace, rename vars etc. do those in another branch, forked from current branch