You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by mp...@apache.org on 2006/01/16 23:47:30 UTC

svn commit: r369598 - in /maven/plugins/trunk/maven-pmd-plugin/src: main/java/org/apache/maven/plugin/pmd/PmdReportListener.java site/apt/howto.apt

Author: mperham
Date: Mon Jan 16 14:47:25 2006
New Revision: 369598

URL: http://svn.apache.org/viewcvs?rev=369598&view=rev
Log:
Remove metrics report as it is noisy and useless, doc formatting

Modified:
    maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReportListener.java
    maven/plugins/trunk/maven-pmd-plugin/src/site/apt/howto.apt

Modified: maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReportListener.java
URL: http://svn.apache.org/viewcvs/maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReportListener.java?rev=369598&r1=369597&r2=369598&view=diff
==============================================================================
--- maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReportListener.java (original)
+++ maven/plugins/trunk/maven-pmd-plugin/src/main/java/org/apache/maven/plugin/pmd/PmdReportListener.java Mon Jan 16 14:47:25 2006
@@ -250,7 +250,12 @@
     {
         sink.section1_();
 
-        processMetrics();
+        // The Metrics report useless with the current PMD metrics impl.
+        // For instance, run the coupling ruleset and you will get a boatload
+        // of excessive imports metrics, none of which is really any use.
+        // TODO Determine if we are going to just ignore metrics.
+        
+        // processMetrics();
 
         sink.body_();
 

Modified: maven/plugins/trunk/maven-pmd-plugin/src/site/apt/howto.apt
URL: http://svn.apache.org/viewcvs/maven/plugins/trunk/maven-pmd-plugin/src/site/apt/howto.apt?rev=369598&r1=369597&r2=369598&view=diff
==============================================================================
--- maven/plugins/trunk/maven-pmd-plugin/src/site/apt/howto.apt (original)
+++ maven/plugins/trunk/maven-pmd-plugin/src/site/apt/howto.apt Mon Jan 16 14:47:25 2006
@@ -10,6 +10,7 @@
   on your project's source code and generate a site report with its results.
 
    * pmd:pmd - run PMD and generate the report
+
    * pmd:cpd - run PMD's Copy/Paste Detector tool and generate a report
 
 Configuration
@@ -37,10 +38,10 @@
                 <artifactId>maven-pmd-plugin</artifactId>
                 <configuration>
                     <rulesets>
-						<ruleset>/rulesets/basic.xml</ruleset>
-						<ruleset>/rulesets/controversial.xml</ruleset>
-						<ruleset>d:\rulesets\strings.xml</ruleset>
-						<ruleset>http://localhost/design.xml</ruleset>
+                        <ruleset>/rulesets/basic.xml</ruleset>
+                        <ruleset>/rulesets/controversial.xml</ruleset>
+                        <ruleset>d:\rulesets\strings.xml</ruleset>
+                        <ruleset>http://localhost/design.xml</ruleset>
                     </rulesets>
                     <format>xml</format>
                     <linkXref>true</linkXref>