buildr-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dona...@apache.org
Subject git commit: Remove the rulesets/java/logging-java.xml ruleset from PMD as it seems premature to use it as a default rule
Date Wed, 18 Jun 2014 07:48:22 GMT
Repository: buildr
Updated Branches:
  refs/heads/master 090b00356 -> 54eb58c3c


Remove the rulesets/java/logging-java.xml ruleset from PMD as it seems premature to use it
as a default rule


Project: http://git-wip-us.apache.org/repos/asf/buildr/repo
Commit: http://git-wip-us.apache.org/repos/asf/buildr/commit/54eb58c3
Tree: http://git-wip-us.apache.org/repos/asf/buildr/tree/54eb58c3
Diff: http://git-wip-us.apache.org/repos/asf/buildr/diff/54eb58c3

Branch: refs/heads/master
Commit: 54eb58c3cbd8cc03fbce03a5a7a71446731cc502
Parents: 090b003
Author: Peter Donald <peter@realityforge.org>
Authored: Wed Jun 18 17:48:11 2014 +1000
Committer: Peter Donald <peter@realityforge.org>
Committed: Wed Jun 18 17:48:11 2014 +1000

----------------------------------------------------------------------
 CHANGELOG           | 4 ++--
 addon/buildr/pmd.rb | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/buildr/blob/54eb58c3/CHANGELOG
----------------------------------------------------------------------
diff --git a/CHANGELOG b/CHANGELOG
index 742f817..48e15a5 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -11,8 +11,8 @@
           configuration within IDEA.
 * Added:  Add ipr.add_java_configuration method for defining a java
           configuration within IDEA.
-* Change: Include additional rules in default pmd rule set: 'logging-java',
-          'finalizers' and 'braces'.
+* Change: Include additional rules in default pmd rule set: 'finalizers' and
+          'braces'.
 * Change: Upgrade the version of PMD in use to 5.1.1.
 * Change: Supply a default xsl file for generating the jdepend report.
 * Fixed:  Fix the naming of css_lint addon file so it is made available

http://git-wip-us.apache.org/repos/asf/buildr/blob/54eb58c3/addon/buildr/pmd.rb
----------------------------------------------------------------------
diff --git a/addon/buildr/pmd.rb b/addon/buildr/pmd.rb
index 5deb842..9e6be27 100644
--- a/addon/buildr/pmd.rb
+++ b/addon/buildr/pmd.rb
@@ -85,7 +85,7 @@ module Buildr
       attr_writer :rule_set_files
 
       def rule_set_files
-        @rule_set_files ||= ['rulesets/java/basic.xml', 'rulesets/java/imports.xml', 'rulesets/java/unusedcode.xml',
'rulesets/java/logging-java.xml', 'rulesets/java/finalizers.xml', 'rulesets/java/braces.xml']
+        @rule_set_files ||= ['rulesets/java/basic.xml', 'rulesets/java/imports.xml', 'rulesets/java/unusedcode.xml',
'rulesets/java/finalizers.xml', 'rulesets/java/braces.xml']
       end
 
       attr_writer :rule_set_paths


Mime
View raw message