The simple way intended to find mistakes overlooked in the initial development phase.
The goal is to improve overall quality of the code .
Practices vary from pair programming, informal walkthroughs, and formal inspection.
OPEN-O uses Gerrit for Code Review
What to review?
- Comply to Coding Guidelines
- Comply to Intellectual Property (Copyright and license headers)
- Comply with Commit Process
- Ensure there is no Chinese Character in comments
Code reviews are expected to take place within the next 24 hours after the code has been submitted within Gerrit.
OPEN-O is an Open Source project and thus anyone who is interested can provide code review.
Best candidate to ensure the code is reviewed are the “Committers” listed in project resources.
Authors are not expected to perform Code Review on their own code.
Tips
Mandatory for all projects, whatever their state.
All Commits must be reviewed.
Depending on the project state, there is a variable depending on the % of committers to perform code review
Feedback provided by voting "-1" (issue), "0" (ok, neutral, abstain), "+1" (approve)
- Incubation: “+1” from 30% of committers (min: 2); no “-1”
- Mature: “+1” from 60% of committers; no “-1”
- Core: “+1” from 75% of committers; no “-1”
The code should not be accepted if there are open issues, but should as soon as the minimum threshold of reviews have come in.