Bump max line length to 140 characters in Checkstyle rules (#1590)

Fixes #465
This commit is contained in:
Yusuke Matsubara 2018-06-05 18:33:51 +09:00 committed by Josephine Lim
parent 32d45b2687
commit 862318c578

View file

@ -7,6 +7,8 @@
Modified from https://github.com/checkstyle/checkstyle/blob/master/src/main/resources/google_checks.xml. Modified from https://github.com/checkstyle/checkstyle/blob/master/src/main/resources/google_checks.xml.
Modifications are: Modifications are:
- doubled each value in Indentation - doubled each value in Indentation
- exceptions for Butter Knife and Espresso
- larger (max)LineLength
Checkstyle configuration that checks the Google coding conventions from Google Java Style Checkstyle configuration that checks the Google coding conventions from Google Java Style
that can be found at https://google.github.io/styleguide/javaguide.html. that can be found at https://google.github.io/styleguide/javaguide.html.
@ -44,7 +46,7 @@
<property name="allowNonPrintableEscapes" value="true"/> <property name="allowNonPrintableEscapes" value="true"/>
</module> </module>
<module name="LineLength"> <module name="LineLength">
<property name="max" value="100"/> <property name="max" value="140"/>
<property name="ignorePattern" value="^package.*|^import.*|a href|href|http://|https://|ftp://"/> <property name="ignorePattern" value="^package.*|^import.*|a href|href|http://|https://|ftp://"/>
</module> </module>
<module name="AvoidStarImport"/> <module name="AvoidStarImport"/>
@ -56,7 +58,7 @@
</module> </module>
<module name="NeedBraces"/> <module name="NeedBraces"/>
<module name="LeftCurly"> <module name="LeftCurly">
<property name="maxLineLength" value="100"/> <property name="maxLineLength" value="140"/>
</module> </module>
<module name="RightCurly"> <module name="RightCurly">
<property name="id" value="RightCurlySame"/> <property name="id" value="RightCurlySame"/>