...
- Please setup your development environment according to ONAP guidelines. Doing so eases the review process for the committers and SME's review your submission.
Sprint Planning User Story Estimations
1 - Extra Small - 1-4 hours
2 - Small - 1 day
3 - Medium - 2-3 days
5 - Large - 1 week
8 - Extra Large - 2-3 weeks
13 - 2x Extra Large - 4-6 weeks
Code Submissions
- All code submissions should not degrade the quality of the policy/* project codebase.
- Any sonar, checkstyle issues must be fixed in the current review or a subsequent review.
- The same JIRA should be used for these submissions.
- API changes most likely will require updates to CSIT tests. It is the responsibility of the contributor to provide a code review to update/enhance the CSIT tests for their contribution.
Java Code Required Practices
The committers have In addition to adhering to ONAP sonar/checkstyle requirements, the project team has agreed upon these desired preferences for submitting java code. WIP
- JUnit Test file naming WIP ( PLD - I thought we had a page on this??)
- WIP
- Use of Static Methods
- WIP
Committer Strategy
- Require 3 committers to +1 a review - at least 2 companies represented.
- Require at least one expert of the repo to +1 (may or may not be a committer).
- For substantial reviews, either size or logic changes, then time limit of 24 hours to review can override.
- A +2 from committer or PTL, one can imply they are good with any future patches for a review.