Skip to content

Latest commit

 

History

History
33 lines (24 loc) · 762 Bytes

File metadata and controls

33 lines (24 loc) · 762 Bytes

See review guidelines in Fraym wiki.

Why

What

Review instructions

Desired feedbacks

  • Readability
  • Correctness (test coverage and beyond)
  • Organization of Code
  • Solution Approach
  • Increase Knowledge Transfer
  • Other (Please detail below)

Developer Checklist

  • 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

Reviewer Checklist

  • I understand what the PR is trying to accomplish
  • Code is readable
  • Tests pass (test suite and any additional instructions)