...
- javadocs, pydocs format preferred on all public methods and classes, classes and modules (files)
Check the code
- Most importantly, does it actually fix what the commitment message says should be fixed?
- Verify against ONAP code standards.
- Java style is specified here, which points to the Google Style Guide repo.
- Javascript code should meet the similar Javascript Google Style Guide, except for the number of characters in one line of code is not restricted.
- Python code should similarly match the Python Google Style Guide.
- Did the unit tests get updated?
- New functionality MUST have unit tests.
- New methods added to existing code MUST be covered by existing or additional unit tests.
- We are looking for a minimum of 60% code coverage, with higher levels encouraged.
...