Skip to content

Commit bedf2f2

Browse files
nielsbasjesnielsbasjes
committed
AVRO-1838: Java: Update checkstyle to catch trailing whitespace. Added missing files.
1 parent f614e15 commit bedf2f2

File tree

3 files changed

+127
-0
lines changed

3 files changed

+127
-0
lines changed

‎dev-tools/pom.xml

+31
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
<?xml version="1.0" encoding="UTF-8"?>
2+
<!--
3+
Licensed to the Apache Software Foundation (ASF) under one or more
4+
contributor license agreements. See the NOTICE file distributed with
5+
this work for additional information regarding copyright ownership.
6+
The ASF licenses this file to You under the Apache License, Version 2.0
7+
(the "License"); you may not use this file except in compliance with
8+
the License. You may obtain a copy of the License at
9+
10+
http://www.apache.org/licenses/LICENSE-2.0
11+
12+
Unless required by applicable law or agreed to in writing, software
13+
distributed under the License is distributed on an "AS IS" BASIS,
14+
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
15+
See the License for the specific language governing permissions and
16+
limitations under the License.
17+
-->
18+
<project
19+
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd"
20+
xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
21+
<modelVersion>4.0.0</modelVersion>
22+
23+
<groupId>org.apache.avro</groupId>
24+
<artifactId>dev-tools</artifactId>
25+
<version>1.9.0-SNAPSHOT</version>
26+
27+
<name>Apache Avro Developer tools</name>
28+
<url>http://avro.apache.org</url>
29+
30+
</project>
31+

‎dev-tools/src/main/resources/checkstyle/checkstyle.xml

+70
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,70 @@
1+
<?xml version="1.0"?>
2+
3+
<!--
4+
Licensed to the Apache Software Foundation (ASF) under one or more
5+
contributor license agreements. See the NOTICE file distributed with
6+
this work for additional information regarding copyright ownership.
7+
The ASF licenses this file to You under the Apache License, Version 2.0
8+
(the "License"); you may not use this file except in compliance with
9+
the License. You may obtain a copy of the License at
10+
11+
http://www.apache.org/licenses/LICENSE-2.0
12+
13+
Unless required by applicable law or agreed to in writing, software
14+
distributed under the License is distributed on an "AS IS" BASIS,
15+
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
16+
See the License for the specific language governing permissions and
17+
limitations under the License.
18+
-->
19+
20+
<!DOCTYPE module PUBLIC
21+
"-//Puppy Crawl//DTD Check Configuration 1.2//EN"
22+
"http://www.puppycrawl.com/dtds/configuration_1_2.dtd">
23+
24+
25+
<module name="Checker">
26+
<module name="SuppressionFilter">
27+
<property name="file" value="${checkstyle.suppressions.file}"/>
28+
</module>
29+
30+
<module name="SuppressionCommentFilter"/>
31+
32+
<module name="FileTabCharacter"/>
33+
<module name="NewlineAtEndOfFile">
34+
<property name="lineSeparator" value="lf"/>
35+
</module>
36+
37+
<module name="TreeWalker">
38+
<module name="ConstantName"/>
39+
<module name="LocalFinalVariableName"/>
40+
<module name="LocalVariableName"/>
41+
<module name="MemberName"/>
42+
<module name="MethodName"/>
43+
<module name="PackageName"/>
44+
<module name="ParameterName"/>
45+
<module name="StaticVariableName"/>
46+
<module name="TypeName"/>
47+
48+
<module name="AvoidStarImport"/>
49+
<module name="RedundantImport"/>
50+
<module name="UnusedImports"/>
51+
52+
<!--<module name="RedundantModifier"/>-->
53+
54+
<module name="EmptyStatement"/>
55+
<module name="IllegalInstantiation"/>
56+
<module name="SimplifyBooleanExpression"/>
57+
<module name="SimplifyBooleanReturn"/>
58+
59+
<module name="InterfaceIsType"/>
60+
61+
<module name="ArrayTypeStyle"/>
62+
<module name="UpperEll"/>
63+
64+
<module name="Regexp">
65+
<property name="format" value="[ \t]+$"/>
66+
<property name="illegalPattern" value="true"/>
67+
<property name="message" value="Trailing whitespace"/>
68+
</module>
69+
</module>
70+
</module>

‎dev-tools/src/main/resources/checkstyle/suppressions.xml

+26
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
<?xml version="1.0"?>
2+
3+
<!--
4+
Licensed to the Apache Software Foundation (ASF) under one or more
5+
contributor license agreements. See the NOTICE file distributed with
6+
this work for additional information regarding copyright ownership.
7+
The ASF licenses this file to You under the Apache License, Version 2.0
8+
(the "License"); you may not use this file except in compliance with
9+
the License. You may obtain a copy of the License at
10+
11+
http://www.apache.org/licenses/LICENSE-2.0
12+
13+
Unless required by applicable law or agreed to in writing, software
14+
distributed under the License is distributed on an "AS IS" BASIS,
15+
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
16+
See the License for the specific language governing permissions and
17+
limitations under the License.
18+
-->
19+
20+
<!DOCTYPE suppressions PUBLIC
21+
"-//Puppy Crawl//DTD Suppressions 1.1//EN"
22+
"http://www.puppycrawl.com/dtds/suppressions_1_1.dtd">
23+
24+
<suppressions>
25+
<suppress files=".*[\\/]target[\\/]" checks=".*" />
26+
</suppressions>

0 commit comments

Comments
 (0)