kemitix/kemitix-checkstyle-ruleset
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

LineLength: set max to 120

Browse source 
Widescreen monitor are more common, and this is the default for IntelliJ. Removing the TODO note about ignoring Javadoc blocks as this is less urget now with the 150% width increase.
This commit is contained in:
Paul Campbell 2016-09-04 22:30:28 +01:00
parent afcbd11e3f
commit c24540c17c
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
README.md
View file

@ -1289,12 +1289,10 @@ class Bar
#### [LineLength](http://checkstyle.sourceforge.net/config_sizes.html#LineLength)
Limits the line length to 80 characters.
Limits the line length to 120 characters.
Doesn't check package or import lines.
> TODO: ignorePattern = "^\s\\* .+" exclude javadoc blocks
#### [LocalFinalVariableName](http://checkstyle.sourceforge.net/config_naming.html#LocalFinalVariableName)
Checks the format of local, `final` variable names, including `catch` parameters.

4
src/main/resources/net/kemitix/checkstyle.xml
View file

@ -100,7 +100,9 @@
<module name="JavaNCSS"/> <!-- Non-Commenting Source Statements complexity analysis -->
<module name="LeftCurly"/> <!-- placement of left curly braces ('{') for code blocks at end of line -->
<module name="LineLength"/> <!-- lines can't be longer the 80 -->
<module name="LineLength"> <!-- lines can't be longer the 120 -->
<property name="max" value="120"/>
</module>
<module name="LocalFinalVariableName"/> <!-- validates identifiers for local, final variables, including catch parameters -->
<module name="LocalVariableName"/> <!-- validates non-final identifiers -->