-
Commits and messages
- Does each commit do exactly one thing?
- Does each commit message concisely describe the one thing the commit does?
- Does each commit message explain any non-obvious changes?
- Could any commits be removed and still achieve the goal of the PR?
- Do command- and instruction-based commits have proper messages?
-
Format and style
- Is this PR about formatting or about behaviour?
- if this PR changes formatting (e.g. whitespace, typos), does it maintain existing behaviour?
- if this PR changes behaviour (e.g. adding a new function), does it maintain formatting?
- Do changes to existing files match the format of surrounding lines?
- Is there documentation for any new or changed behaviour?
- Is this PR about formatting or about behaviour?
-
Notifications