Skip to content

Commit f1ef3d1

Browse files
committed
HV-??? Remove checkstyle rules addressed by other plugins
1 parent a412f6b commit f1ef3d1

File tree

1 file changed

+5
-62
lines changed

1 file changed

+5
-62
lines changed

build/build-config/src/main/resources/checkstyle.xml

Lines changed: 5 additions & 62 deletions
Original file line numberDiff line numberDiff line change
@@ -14,69 +14,24 @@
1414

1515
<module name="TreeWalker">
1616

17+
<!-- cannot do with spotless but then is it really needed ? -->
1718
<module name="RegexpSinglelineJava">
1819
<!-- do not allow a package declaration that contains ".target." or "target;" -->
1920
<property name="format" value="^package .*\.target[\.;]"/>
2021
<property name="message" value="Do not use &quot;target&quot; as package name element"/>
2122
</module>
2223

23-
<!-- Use tabs -->
24-
<module name="RegexpSinglelineJava">
25-
<property name="format" value="^ +\t*\S" />
26-
<property name="message" value="Line has leading space characters; indentation should be performed with tabs only." />
27-
<property name="ignoreComments" value="true" />
28-
</module>
29-
30-
<!-- Check parentheses padding -->
31-
<module name="RegexpSinglelineJava">
32-
<property name="format" value="^\t+(if|for|while|catch)([^ ])\(" />
33-
<property name="message" value="Left parentheses should be preceeded by 1 whitespace" />
34-
<property name="ignoreComments" value="true" />
35-
</module>
36-
37-
<module name="RegexpSinglelineJava">
38-
<property name="format" value="^\t+(if|for|while) \( .+[^ ]\) \{$" />
39-
<property name="message" value="Right parentheses should be preceeded by 1 white space" />
40-
<property name="ignoreComments" value="true" />
41-
</module>
42-
43-
<module name="RegexpSinglelineJava">
44-
<property name="format" value="^\t+(if|for|while) \([^ $]" />
45-
<property name="message" value="Left parentheses should be followed by 1 whitespace" />
46-
<property name="ignoreComments" value="true" />
47-
</module>
48-
49-
<module name="RegexpSinglelineJava">
50-
<property name="format" value="\)([^ ])\{$" />
51-
<property name="message" value="Right parentheses should be followed by 1 white space" />
52-
<property name="ignoreComments" value="true" />
53-
</module>
54-
55-
<module name="RegexpSinglelineJava">
56-
<property name="format" value="catch \([^\w]" />
57-
<property name="message" value="'catch' clause does not require white space after left parentheses" />
58-
<property name="ignoreComments" value="true" />
59-
</module>
60-
61-
<module name="RegexpSinglelineJava">
62-
<property name="format" value="catch \(.*\s\) \{" />
63-
<property name="message" value="'catch' clause does not require white space before right parentheses" />
64-
<property name="ignoreComments" value="true" />
65-
</module>
66-
67-
<module name="RegexpSinglelineJava">
68-
<property name="format" value="\)\s+;$" />
69-
<property name="message" value="Unecessary whitespace after parentheses" />
70-
<property name="ignoreComments" value="true" />
71-
</module>
7224

25+
<!-- cannot do with spotless, there's a workaround to use regex-replace -->
7326
<!-- Checks for imports -->
7427
<module name="AvoidStarImport"/>
7528

7629
<!-- Checks for common coding problems -->
30+
<!-- didn't find any alternatives for these two: -->
7731
<module name="EqualsHashCode" />
7832
<module name="IllegalInstantiation" />
7933

34+
<!-- didn't find any alternatives for this next group: -->
8035
<!-- Miscellaneous other checks. -->
8136
<module name="ModifierOrder" />
8237
<module name="PackageAnnotation" />
@@ -93,18 +48,14 @@
9348
<module name="EqualsAvoidNull" />
9449
<module name="UpperEll"/>
9550

51+
<!-- didn't find any alternatives for this next group: -->
9652
<!-- not checking for Constructor here: -->
9753
<module name="RedundantModifier">
9854
<property name="tokens"
9955
value="METHOD_DEF, VARIABLE_DEF, ANNOTATION_FIELD_DEF, INTERFACE_DEF, CLASS_DEF, ENUM_DEF"/>
10056
</module>
10157
<module name="MissingOverride" />
10258

103-
<module name="MethodParamPad">
104-
<property name="option" value="nospace" />
105-
<property name="allowLineBreaks" value="false" />
106-
</module>
107-
10859
<!-- Checks for blocks. You know, those {}'s, not doing this for methods because of 1 line getters/setters-->
10960
<module name="LeftCurly">
11061
<property name="option" value="eol"/>
@@ -130,16 +81,8 @@
13081
<module name="SuppressionCommentFilter" />
13182
</module>
13283

133-
<module name="RegexpSingleline">
134-
<property name="format" value="\s+$" />
135-
<property name="message" value="White spaces at the end of line" />
136-
</module>
137-
13884
<module name="SuppressionFilter">
13985
<property name="file" value="/suppressions.xml" />
14086
</module>
14187

142-
<!-- Checks that a file ends with a new line -->
143-
<module name="NewlineAtEndOfFile"/>
144-
14588
</module>

0 commit comments

Comments
 (0)