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 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