Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

[MCHECKSTYLE-325] Provided multi-module IT with verification. Provide… #88

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from
Open

[MCHECKSTYLE-325] Provided multi-module IT with verification. Provide… #88

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
10f735a
[MCHECKSTYLE-325] Provided multi-module IT with verification. Provide…
lennartj Jul 5, 2016
a176f23
[MCHECKSTYLE-287] Introduced null checks, and logged attempts to use …
lennartj Jul 5, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 34 additions & 0 deletions maven-checkstyle-plugin/src/it/multimodule-configlocation/codestyle1/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
<?xml version="1.0" encoding="UTF-8"?>
<!--
~ Licensed to the Apache Software Foundation (ASF) under one
~ or more contributor license agreements. See the NOTICE file
~ distributed with this work for additional information
~ regarding copyright ownership. The ASF licenses this file
~ to you under the Apache License, Version 2.0 (the
~ "License"); you may not use this file except in compliance
~ with the License. You may obtain a copy of the License at
~
~ http://www.apache.org/licenses/LICENSE-2.0
~
~ Unless required by applicable law or agreed to in writing,
~ software distributed under the License is distributed on an
~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
~ KIND, either express or implied. See the License for the
~ specific language governing permissions and limitations
~ under the License.
-->
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
<modelVersion>4.0.0</modelVersion>

<parent>
<groupId>se.west.foobar</groupId>
<artifactId>foobar-parent</artifactId>
<version>1.0.0-SNAPSHOT</version>
</parent>

<groupId>se.west.foobar.codestyle1</groupId>
<artifactId>foobar-codestyle1</artifactId>
<packaging>jar</packaging>

</project>
27 changes: 27 additions & 0 deletions ...odule-configlocation/codestyle1/src/main/resources/checkstyle/checkstyle-suppressions.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
<?xml version="1.0" encoding="UTF-8"?>
<!--
Licensed to the Apache Software Foundation (ASF) under one
or more contributor license agreements. See the NOTICE file
distributed with this work for additional information
regarding copyright ownership. The ASF licenses this file
to you under the Apache License, Version 2.0 (the
"License"); you may not use this file except in compliance
with the License. You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing,
software distributed under the License is distributed on an
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
KIND, either express or implied. See the License for the
specific language governing permissions and limitations
under the License.
-->
<!DOCTYPE suppressions PUBLIC "-//Puppy Crawl//DTD Suppressions 1.0//EN"
"http://www.puppycrawl.com/dtds/suppressions_1_0.dtd">
<suppressions>
<!--
No default suppressions, this file is simply a placeholder
to enable custom suppressions in child reactors.
-->
</suppressions>
170 changes: 170 additions & 0 deletions ...src/it/multimodule-configlocation/codestyle1/src/main/resources/checkstyle/checkstyle.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,170 @@
<?xml version="1.0" encoding="UTF-8"?>
<!--
Licensed to the Apache Software Foundation (ASF) under one
or more contributor license agreements. See the NOTICE file
distributed with this work for additional information
regarding copyright ownership. The ASF licenses this file
to you under the Apache License, Version 2.0 (the
"License"); you may not use this file except in compliance
with the License. You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing,
software distributed under the License is distributed on an
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
KIND, either express or implied. See the License for the
specific language governing permissions and limitations
under the License.
-->
<!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.3//EN"
"http://www.puppycrawl.com/dtds/configuration_1_3.dtd">

<module name="Checker">
<property name="severity" value="warning"/>

<!--
Permit use of code annotations to suppress warnings.
-->
<module name="SuppressWarningsFilter"/>

<module name="TreeWalker">

<!--
Permit use of code annotations to suppress warnings.
-->
<module name="SuppressWarningsHolder" />
<module name="JavadocMethod">
<property name="scope" value="public"/>
<property name="allowMissingPropertyJavadoc" value="true"/>
<property name="tokens" value="METHOD_DEF"/>
<property name="allowUndeclaredRTE" value="true"/>
</module>
<module name="LineLength">
<property name="max" value="120"/>
</module>

<module name="JavadocStyle">
<property name="checkFirstSentence" value="false"/>
<property name="checkHtml" value="false"/>
</module>

<module name="ConstantName">
<property name="format" value="(^[A-Z][A-Z0-9_]*$)|log"/>
</module>
<module name="LocalFinalVariableName">
<property name="format" value="(^[a-z][a-zA-Z0-9]*$)|(^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$)"/>
</module>
<module name="LocalVariableName"/>
<module name="MemberName"/>
<module name="MethodName"/>
<module name="PackageName"/>
<module name="ParameterName"/>
<module name="StaticVariableName"/>
<module name="TypeName"/>
<module name="AvoidStarImport">
<property name="severity" value="error"/>
</module>
<module name="JavaNCSS"/>

<!--
Enforce correct logging mechanics in release artifacts.
-->
<module name="IllegalImport">
<property name="severity" value="error"/>
<property name="illegalPkgs"
value="java.util.logging, org.jboss.logging, org.apache.log4j, org.apache.commons.logging"/>
</module>

<!--
We should not use Calendar or Date types in any public method.
They could be used internally - and only within model classes to cope with JPA time types -
but not within parameters or return types.
-->
<module name="IllegalType">
<property name="severity" value="error"/>
<property name="tokens" value="PARAMETER_DEF, METHOD_DEF"/>
<property name="illegalClassNames"
value="java.util.GregorianCalendar, java.util.Calendar, java.util.Date, java.util.Vector, java.util.Properties"/>
</module>

<!--
Validate that we don't use the System.exit, out or err in the code.
Use proper logging instead.
-->
<module name="Regexp">
<property name="message" value="System.exit"/>
<property name="severity" value="error"/>
<property name="format" value="System\.exit"/>
<property name="illegalPattern" value="true"/>
</module>
<module name="Regexp">
<property name="message" value="System.out"/>
<property name="severity" value="error"/>
<property name="format" value="System\.out."/>
<property name="illegalPattern" value="true"/>
</module>
<module name="Regexp">
<property name="message" value="System.err"/>
<property name="severity" value="error"/>
<property name="format" value="System\.err."/>
<property name="illegalPattern" value="true"/>
</module>

<module name="RedundantImport"/>
<module name="UnusedImports"/>
<module name="MethodLength"/>
<module name="ParameterNumber">
<property name="max" value="5"/>
<property name="tokens" value="METHOD_DEF"/>
</module>

<module name="EmptyForIteratorPad"/>
<module name="MethodParamPad"/>
<module name="OperatorWrap"/>
<module name="ParenPad"/>
<module name="TypecastParenPad"/>
<module name="WhitespaceAfter"/>
<module name="ModifierOrder"/>
<module name="AvoidNestedBlocks"/>
<module name="EmptyBlock">
<property name="option" value="text"/>
</module>
<module name="LeftCurly"/>
<module name="NeedBraces"/>
<module name="RightCurly"/>
<module name="EmptyStatement"/>
<module name="EqualsHashCode"/>
<module name="IllegalInstantiation"/>
<module name="InnerAssignment"/>
<module name="MagicNumber"/>
<module name="MissingSwitchDefault"/>
<module name="SimplifyBooleanExpression"/>
<module name="SimplifyBooleanReturn"/>
<module name="FinalClass"/>
<module name="HideUtilityClassConstructor"/>
<module name="InterfaceIsType"/>
<module name="VisibilityModifier">
<property name="packageAllowed" value="true"/>
<property name="protectedAllowed" value="true"/>
</module>
<module name="ArrayTypeStyle"/>
<module name="FinalParameters"/>
<module name="UpperEll"/>
<module name="JavadocType">
<property name="excludeScope" value="private"/>
<property name="scope" value="public"/>
<property name="authorFormat" value="\S"/>
<property name="tokens" value="INTERFACE_DEF"/>
</module>
<module name="FallThrough"/>
<module name="DefaultComesLast"/>
<module name="ReturnCount">
<property name="format" value="^equals$|^compareTo$"/>
<property name="max" value="3"/>
</module>
</module>
<module name="FileLength"/>
<module name="FileTabCharacter"/>
<module name="Translation"/>
</module>
34 changes: 34 additions & 0 deletions maven-checkstyle-plugin/src/it/multimodule-configlocation/codestyle2/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
<?xml version="1.0" encoding="UTF-8"?>
<!--
~ Licensed to the Apache Software Foundation (ASF) under one
~ or more contributor license agreements. See the NOTICE file
~ distributed with this work for additional information
~ regarding copyright ownership. The ASF licenses this file
~ to you under the Apache License, Version 2.0 (the
~ "License"); you may not use this file except in compliance
~ with the License. You may obtain a copy of the License at
~
~ http://www.apache.org/licenses/LICENSE-2.0
~
~ Unless required by applicable law or agreed to in writing,
~ software distributed under the License is distributed on an
~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
~ KIND, either express or implied. See the License for the
~ specific language governing permissions and limitations
~ under the License.
-->
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
<modelVersion>4.0.0</modelVersion>

<parent>
<groupId>se.west.foobar</groupId>
<artifactId>foobar-parent</artifactId>
<version>1.0.0-SNAPSHOT</version>
</parent>

<groupId>se.west.foobar.codestyle2</groupId>
<artifactId>foobar-codestyle2</artifactId>
<packaging>jar</packaging>

</project>
27 changes: 27 additions & 0 deletions ...odule-configlocation/codestyle2/src/main/resources/checkstyle/checkstyle-suppressions.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
<?xml version="1.0" encoding="UTF-8"?>
<!--
Licensed to the Apache Software Foundation (ASF) under one
or more contributor license agreements. See the NOTICE file
distributed with this work for additional information
regarding copyright ownership. The ASF licenses this file
to you under the Apache License, Version 2.0 (the
"License"); you may not use this file except in compliance
with the License. You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing,
software distributed under the License is distributed on an
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
KIND, either express or implied. See the License for the
specific language governing permissions and limitations
under the License.
-->
<!DOCTYPE suppressions PUBLIC "-//Puppy Crawl//DTD Suppressions 1.0//EN"
"http://www.puppycrawl.com/dtds/suppressions_1_0.dtd">
<suppressions>
<!--
No default suppressions, this file is simply a placeholder
to enable custom suppressions in child reactors.
-->
</suppressions>
Loading