Reviewer Check List

Jump to bottom

M Starch edited this page Jul 21, 2022 · 2 revisions

## Pre-CI Checks: Must perform before launching CI
- [ ] No obviously malicious code introduced
- [ ] PR is against `devel`. Use `Edit -> change branch` to fix, or close PR.

## Code Review
- [ ] Code follows flight software quality guidelines
- [ ] Changes are of reasonable length for review

## Procedural Checks
- [ ] PR passes CI
- [ ] Accept with `squash-and-merge`

Clone this wiki locally