Details
-
Task
-
Status: Open
-
P3
-
Resolution: Unresolved
-
None
-
None
-
None
Description
Checkstyle has a rule (EmptyLineSeparator) we currently don't use to enforce the use of blank lines in various places. In order to reduce the noise in reviews of PR's, I like to start using it.
This will likely be accomplished via a series of PR's. Just adding the default EmptyLineSeparator to the checkstyle rules results in a THOUSANDS of violations which would be hard to review. I will likely enable it for one token type at a time, fix the violations on that particular token type, submit, add then next token type, etc... That should (hopefully) keep each PR down to something that can reasonably be reviewed.