Details
-
Improvement
-
Status: Open
-
Trivial
-
Resolution: Unresolved
-
None
-
None
-
None
Description
Improve the existing checkstyle code convention to make it easier to accept pull requests from the community and ensure the codebase has a consistent feel.
Use Accumulo's ruleset as a guide: https://github.com/apache/accumulo/blob/master/pom.xml
- Include a check to ensure classes with @Test annotations are suffixed with IT or Test. Also make the class name suffix *IntegrationTest illegal.
- Include a check to ensure we don't use the JUnit 3 API which has some deprecated classes.
- Include a check to ensure we're not using an incorrect Apache header with a copyright clause.
Attachments
Issue Links
- is related to
-
RYA-358 Add checkstyle rules to build
- Resolved