- The Goal of Review: Code reviews exist to ensure reproducibility, readability, and shared understanding, not to nitpick style.
- Automated Checks First: Before requesting a review, ensure CI passes (linting, formatting, and unit tests). Do not use human review time for things machines can catch.
- Review Process:
- Keep PRs small and focused on a single logical change.
- Use PR Templates to ensure consistent descriptions that explain why the change was made and how to test it.
- Reviewers should verify the code against the acceptance criteria and metrics defined in the task.