diff --git a/Dangerfile b/Dangerfile index 0d95d7712c..07a9aef9a2 100644 --- a/Dangerfile +++ b/Dangerfile @@ -1,4 +1,4 @@ -require 'pmdtester/runner' +require 'pmdtester' require 'time' require 'logger' @@ -6,14 +6,15 @@ require 'logger' def run_pmdtester Dir.chdir('..') do - argv = ['-r', './pmd', '-b', "#{ENV['TRAVIS_BRANCH']}", '-p', 'FETCH_HEAD', '-m', 'online'] + argv = ['-r', './pmd', '-b', "#{ENV['TRAVIS_BRANCH']}", '-p', 'FETCH_HEAD', '-m', 'online', '-a'] Process.fork do begin runner = PmdTester::Runner.new(argv) - runner.run # Optimize: get more information from runner, e.g. introduce new pmd errors? + introduce_new_pmd_errors = runner.run + warn("The PR may introduce new PMD errors!") if introduce_new_pmd_errors rescue StandardError => e - @logger.error "Running pmdtester failed: #{e.msg}" warn("Running pmdtester failed, this message is mainly used to remind the maintainers of PMD.") + @logger.error "Running pmdtester failed: #{e.inspect}" exit 1 end end @@ -37,7 +38,6 @@ def upload_report @logger.info "Successfully uploaded #{tar_filename} to chunk.io" # set value of sticky to true and the message is kept after new commits are submited to the PR - message("Please check the [regression diff report](#{report_url.chomp}/diff/index.html) to make sure that everything is expected", sticky: true) else @logger.error "Error while uploading #{tar_filename} to chunk.io: #{report_url}" diff --git a/docs/pages/release_notes.md b/docs/pages/release_notes.md index fa5dea7182..d67d2ed769 100644 --- a/docs/pages/release_notes.md +++ b/docs/pages/release_notes.md @@ -40,3 +40,4 @@ This is a minor release. * [#1254](https://github.com/pmd/pmd/pull/1254): \[ci] \[GSoC] Integrating the danger and pmdtester to travis CI - [BBG](https://github.com/djydewang) * [#1258](https://github.com/pmd/pmd/pull/1258): \[java] Use typeof in MissingSerialVersionUID - [krichter722](https://github.com/krichter722) * [#1264](https://github.com/pmd/pmd/pull/1264): \[cpp] Fix NullPointerException in CPPTokenizer:99 - [Rafael Cortês](https://github.com/mrfyda) +* [#1278](https://github.com/pmd/pmd/pull/1278): \[ci] \[GSoC] Use pmdtester 1.0.0.pre.beta3 - [BBG](https://github.com/djydewang)