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

MethodLength: reduce limit to 40 lines

Browse source
This commit is contained in:
Paul Campbell 2016-09-05 17:30:42 +01:00
parent 02a755f378
commit 633b270172
2 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
README.md
View file

@ -1320,8 +1320,6 @@ Restricts the number of methods in a type to 30.
Restricts the number of lines in a method to 60. Include blank lines and single line comments. You should be able to see an entire method without needing to scroll. Restricts the number of lines in a method to 60. Include blank lines and single line comments. You should be able to see an entire method without needing to scroll.
> TODO: max = 40
#### [MethodName](http://checkstyle.sourceforge.net/config_naming.html#MethodName) #### [MethodName](http://checkstyle.sourceforge.net/config_naming.html#MethodName)
Checks the format of method names. Checks the format of method names.

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

@ -126,7 +126,7 @@
<property name="maxTotal" value="30"/> <property name="maxTotal" value="30"/>
</module> </module>
<module name="MethodLength"> <!-- restrict the number of lines in a method --> <module name="MethodLength"> <!-- restrict the number of lines in a method -->
<property name="max" value="60"/> <property name="max" value="40"/>
</module> </module>
<module name="MethodName"/> <!-- method names conform to ^[a-z][a-zA-Z0-9]*$ --> <module name="MethodName"/> <!-- method names conform to ^[a-z][a-zA-Z0-9]*$ -->
<module name="MethodParamPad"/> <!-- verifies padding around method parameters --> <module name="MethodParamPad"/> <!-- verifies padding around method parameters -->