Merge branch 'pr-30'

Author: adangel

History.md

@@ -3,6 +3,7 @@
 ## External Contributions
 
 *   [#28](https://github.com/pmd/pmd-regression-tester/pull/28): Refactor require statements - [BBG](https://github.com/djydewang)
+*   [#30](https://github.com/pmd/pmd-regression-tester/pull/30): Fix diff_cmd in RuleSetBuilder - [BBG](https://github.com/djydewang)
 
 # 1.0.0.beta2 / 2018-07-17
 

lib/pmdtester/builders/rule_set_builder.rb

@@ -46,7 +46,7 @@ def diff_filenames
         base = @options.base_branch
         patch = @options.patch_branch
         # We only need to support git here, since PMD's repo is using git.
-        diff_cmd = "git diff --name-only #{base}..#{patch} -- pmd/core pmd/java"
+        diff_cmd = "git diff --name-only #{base}..#{patch} -- pmd-core pmd-java"
         filenames = Cmd.execute(diff_cmd)
       end
       filenames.split("\n")
@@ -78,7 +78,7 @@ def get_rule_sets(filenames)
 
     def build_config_file(rule_sets)
       if rule_sets.empty?
-        puts NO_JAVA_RULES_CHANGED_MESSAGE
+        logger.info NO_JAVA_RULES_CHANGED_MESSAGE
         exit 0
       end