You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by ca...@apache.org on 2005/09/30 01:01:27 UTC

svn commit: r292574 - /maven/maven-1/plugins/trunk/checkstyle/plugin.jelly

Author: carlos
Date: Thu Sep 29 16:01:26 2005
New Revision: 292574

URL: http://svn.apache.org/viewcvs?rev=292574&view=rev
Log:
Take checkstyle:report-internal from checkstyle:run
Improve docs

Modified:
    maven/maven-1/plugins/trunk/checkstyle/plugin.jelly

Modified: maven/maven-1/plugins/trunk/checkstyle/plugin.jelly
URL: http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/checkstyle/plugin.jelly?rev=292574&r1=292573&r2=292574&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/checkstyle/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/checkstyle/plugin.jelly Thu Sep 29 16:01:26 2005
@@ -39,20 +39,16 @@
     <assert:assertPluginAvailable groupId="maven" artifactId="maven-xdoc-plugin" minRelease="1.10" neededBy="${plugin.artifactId}"/>
   </j:if>
 
-  <!--
-     ========================================================================
-       Default goal.
-     ========================================================================
-  -->
-  <goal name="checkstyle" description="Perform checkstyle checks"
+
+  <!-- ======================================================================== -->
+
+  <goal name="checkstyle" description="Call checkstyle:report"
       prereqs="checkstyle:report"/>
 
-  <!--
-     ========================================================================
-       Initializations.
-     ========================================================================
-  -->
-  <goal name="checkstyle:init">
+
+  <!-- ======================================================================== -->
+
+  <goal name="checkstyle:init" description="Initialize plugin">
 
     <j:if test="${sourcesPresent == 'true'}">
 
@@ -97,14 +93,11 @@
 
   </goal>
 
-  <!--
-     ========================================================================
-       Generate a checkstyle report by running checkstyle on the project
-       source code.
-     ========================================================================
-  -->
+
+  <!-- ======================================================================== -->
+
   <goal name="checkstyle:report"
-    description="Perform checkstyle checks"
+    description="Run checkstyle on project source code and generate reports"
     prereqs="checkstyle:init">
 
     <j:if test="${sourcesPresent == 'true'}">
@@ -132,11 +125,15 @@
       </j:if>
 
       <attainGoal name="checkstyle:run"/>
+      <attainGoal name="checkstyle:report-internal"/>
     </j:if>
     
   </goal>
 
-  <goal name="checkstyle:run">
+
+  <!-- ======================================================================== -->
+
+  <goal name="checkstyle:run" description="Run checkstyle on the sources and generate the xml report">
 
     <!-- Create the directory for the checkstyle cache in case it doesn't 
          exist already -->
@@ -152,7 +149,7 @@
           </j:when>
           <j:otherwise>
             <echo>The locale defined in maven.xdoc.locale.default is invalid.</echo>
-            <echo>As Coutry code we'll use EN instead.</echo>
+            <echo>As Country code we'll use EN instead.</echo>
             <j:set var="localeCountry" value="EN"/>
           </j:otherwise>
     </j:choose>
@@ -205,7 +202,6 @@
         </j:forEach>
       </j:if>
 
-
       <ant:classpath>
           <ant:path refid="maven.dependency.classpath"/>
           <ant:pathelement path="${maven.build.dest}"/>
@@ -225,7 +221,14 @@
       </j:choose>
 
     </ant:checkstyle>
-      
+
+  </goal>
+
+
+  <!-- ======================================================================== -->
+
+  <goal name="checkstyle:report-internal" description="Generate xdocs from checkstyle xml report">
+
     <doc:jsl
       input="${maven.build.dir}/checkstyle-raw-report.xml"
       output="checkstyle-report.xml"
@@ -254,12 +257,10 @@
 
   </goal>
 
-  <!--
-     ========================================================================
-       Register a Checkstyle report to the "site" plugin.
-     ========================================================================
-  -->
-  <goal name="maven-checkstyle-plugin:register">
+
+  <!-- ======================================================================== -->
+
+  <goal name="maven-checkstyle-plugin:register" description="Register a Checkstyle report to the site plugin">
     <j:if test="${sourcesPresent == 'true'}">
       <doc:registerReport 
         name="Checkstyle" 
@@ -269,17 +270,17 @@
     </j:if>
   </goal>
   
-  <!--
-     ========================================================================
-       Deregister a Checkstyle report to the "site" plugin.
-     ========================================================================
-  -->
-  <goal name="maven-checkstyle-plugin:deregister">
+
+  <!-- ======================================================================== -->
+
+  <goal name="maven-checkstyle-plugin:deregister" description="Deregister a Checkstyle report to the site plugin.">
     <j:if test="${sourcesPresent == 'true'}">
       <doc:deregisterReport name="Checkstyle"/>
     </j:if>
   </goal>
 
+
+  <!-- ======================================================================== -->
 
   <!-- Generate a report filtering errors by severity -->
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org