-
Notifications
You must be signed in to change notification settings - Fork 1
CodeReview
Andy Porter edited this page May 2, 2018
·
6 revisions
Before a pull request can be merged to master it must pass a code review. The rules for a code review are as follows:
- Branch must compile without compiler warnings
- Reviewer must examine the code changes (using the 'diff' tab on the PR) and check that any new code is:
- Correct
- Well commented
- As easy to understand as possible
- Documented using Doxygen markup