Repository: buildr
Updated Branches:
  refs/heads/master 533fe2be0 -> 2e664469e


Include additional rules in default pmd rule set: 'logging-java', 'finalizers' 
and 'braces'. These rules seem nearly universal


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

Branch: refs/heads/master
Commit: 2e664469e29b25eddfbbb81030c89f32d7c9180d
Parents: 533fe2b
Author: Peter Donald <[email protected]>
Authored: Fri May 30 11:21:51 2014 +1000
Committer: Peter Donald <[email protected]>
Committed: Fri May 30 11:21:51 2014 +1000

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


http://git-wip-us.apache.org/repos/asf/buildr/blob/2e664469/CHANGELOG
----------------------------------------------------------------------
diff --git a/CHANGELOG b/CHANGELOG
index bbf4851..d2e479e 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,4 +1,6 @@
 1.4.18 (Pending)
+* Change: Include additional rules in default pmd rule set: 'logging-java',
+          '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/2e664469/addon/buildr/pmd.rb
----------------------------------------------------------------------
diff --git a/addon/buildr/pmd.rb b/addon/buildr/pmd.rb
index 8c2a7bd..5deb842 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']
+        @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']
       end
 
       attr_writer :rule_set_paths

Reply via email to