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