Apache
/
activemq-artemis
mirror of https://github.com/apache/activemq-artemis.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use sevntu-checkstyle check for required annotation parameters 

RequiredParameterForAnnotation available on Sevntu since https://github.com/sevntu-checkstyle/sevntu.checkstyle/issues/321
This commit is contained in:
Ville Skyttä 2016-06-14 00:34:55 +03:00 committed by Clebert Suconic
parent 33a51223b0
commit b3ed6fbd43
2 changed files with 4 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
etc/checkstyle.xml
View File

@ -94,13 +94,6 @@ under the License.
<property name="illegalPkgs" value="junit.framework"/>
</module>
<!-- TODO: Where to place this now? -->
<!-- developed at https://github.com/hornetq/hornetq-checkstyle-checks -->
<module name="org.hornetq.checks.annotation.RequiredAnnotation">
<property name="annotationName" value="Parameters"/>
<property name="requiredParameters" value="name"/>
</module>
<module name="WriteTag">
<property name="tag" value="@author"/>
<property name="tagSeverity" value="error"/>
@ -109,6 +102,10 @@ under the License.
<!-- Sevntu checks, http://sevntu-checkstyle.github.io/sevntu.checkstyle/ -->
<module name="DiamondOperatorForVariableDefinition"/>
<module name="RequiredParameterForAnnotation">
<property name="annotationName" value="Parameterized.Parameters"/>
<property name="requiredParameters" value="name"/>
</module>
</module>
</module>

8
pom.xml
View File

@ -1067,14 +1067,6 @@
<artifactId>sevntu-checkstyle-maven-plugin</artifactId>
<version>1.21.0</version>
</dependency>
<!-- This was initially done to enforce name on Parameter annotation
I've developed a customized check and I needed this jar to deploy the specialized checkstyle -->
<dependency>
<!-- TODO Find a better home -->
<groupId>org.hornetq</groupId>
<artifactId>hornetq-checkstyle-checks</artifactId>
<version>0.2</version>
</dependency>
</dependencies>
<configuration>
<skip>${skipStyleCheck}</skip>