You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by de...@apache.org on 2009/07/10 08:45:44 UTC

svn commit: r792817 - /maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml

Author: dennisl
Date: Fri Jul 10 06:45:44 2009
New Revision: 792817

URL: http://svn.apache.org/viewvc?rev=792817&view=rev
Log:
[MCHECKSTYLE-98] Maven Checkstyle is too strict and not follow Maven's team conventions!
Submitted by: Vincent Siveton
Reviewed by: Dennis Lundberg

o Partially applied, see comments in issue for details

Modified:
    maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml

Modified: maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml?rev=792817&r1=792816&r2=792817&view=diff
==============================================================================
--- maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml (original)
+++ maven/plugins/trunk/maven-checkstyle-plugin/src/main/resources/config/maven_checks.xml Fri Jul 10 06:45:44 2009
@@ -65,10 +65,14 @@
         <!-- See http://checkstyle.sf.net/config_javadoc.html -->
         <module name="JavadocMethod">
           <property name="severity" value="warning"/>
+          <property name="scope" value="protected"/>
+        </module>
+        <module name="JavadocType">
+          <property name="scope" value="protected"/>
         </module>
-        <module name="JavadocType"/>
         <module name="JavadocVariable">
           <property name="severity" value="info"/>
+          <property name="scope" value="protected"/>
         </module>
 
 
@@ -167,6 +171,7 @@
         <module name="InterfaceIsType"/>
         <module name="VisibilityModifier">
           <property name="protectedAllowed" value="true"/>
+          <property name="packageAllowed" value="true"/>
         </module>
 
         <!-- Miscellaneous other checks.                   -->