892 B
892 B
Thanks for submitting a PR!
Code Review Checklist (to be filled out by reviewer)
- Does the PR describe what changes are being made?
- Does the PR describe why the changes are being made?
- Does the code follow our style guide?
- Does the code follow our testing guide?
- Could the PR be broken into smaller PRs?
- Does the new code have enough tests? (every PR should have tests or justification otherwise. Bug-fix PRs especially)
- Does the new code have enough documentation that answers "how do I use it?" and "what does it do?"? (both source documentation and higher level, diagrams?)
- Does any documentation need updating?