The Development Procedures and Policies section of this wiki documents the general procedures and policies for submitting code to ONAP projects. For the benefit of CCSDK project submitters, we ask contributors to follow the following specific guidelines, in line with best practices for most other ONAP projects.
Guidelines for contributors:
- Each code submission MUST include the JIRA identifier of the issue being addressed by this change. That issue is specified in the Issue-ID footer, as documented in the ONAP development procedures and policies page noted above.
- Each code submission that includes new code branches MUST include automated jUnit test cases to exercise the new code branches added.
- The groupId in each pom.xml must match the repository name defined in Gerrit. For example, all projects in ccsdk/features must have groupIds that begin with org.onap.ccsdk.features
Guidelines for committers/PTL:
- When performing code reviews, committers should note whether the submission contains appropriate test cases. If the committer believes this submission could reduce the projects' code coverage, he/she should vote -1 on the submission and add a comment advising the submitter that they need to add appropriate testing.
- After code is merged, the PTL will review code coverage in Sonar. If the merge reduces the unit test coverage, the PTL will ask the developer who submitted that change to submit a new change adding appropriate testing.
- After code is merged, the committer who merged the code should verify that the merge build was successful in Jenkins.