-
Notifications
You must be signed in to change notification settings - Fork 0
definition_done
Antonio Robirosa edited this page Feb 12, 2020
·
3 revisions
- Code compiles
- All old tests are working
- No PMD, Spotbugs or Checkstyle violations excluding the training extensions with the standard code of the accelerator.
- The test coverage is equal or higher to 70% of the lines
- The new tests are meaningful
- Code is committed in the repository and no files or changes are missing
- The change was tested
- The feature is documented on the wiki
- The feature branch is merged in master
- The ticket is closed