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.
ONAP uses Gerrit for Code Review
What to review?
- Comply to Coding Guidelines: see Java code style
- 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 36 business hours after the code has been submitted within Gerrit.
ONAP 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. This rule is especially true for integration work.
See also Committer Best Practices
| Tip | ||||
|---|---|---|---|---|
| ||||
Mandatory for all projects, whatever their state. Self-commits are not allowed All Commits must be reviewed and integration tested. 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)
The code should not be accepted if there are open issues, but should as soon as the minimum threshold of reviews have come in. |