See review guidelines in Fraym wiki.
- Readability
- Correctness (test coverage and beyond)
- Organization of Code
- Solution Approach
- Increase Knowledge Transfer
- Other (Please detail below)
- I've added documentation where possible
- I've run pre-commit hooks
- I've added unit tests where possible
- I've added inline Github comments to point out notable code changes or justifications for decisions
- I understand what the PR is trying to accomplish
- Code is readable
- Tests pass (test suite and any additional instructions)