Skip to content

Commit

Permalink
Danger: Update PmdTester execution to report also configuration errors
Browse files Browse the repository at this point in the history
  • Loading branch information
adangel committed Jan 20, 2019
1 parent dbe2442 commit ecd60e8
Showing 1 changed file with 10 additions and 7 deletions.
17 changes: 10 additions & 7 deletions Dangerfile
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ def run_pmdtester
argv = ['-r', './pmd', '-b', "#{ENV['TRAVIS_BRANCH']}", '-p', 'FETCH_HEAD', '-m', 'online', '-a']
begin
runner = PmdTester::Runner.new(argv)
@new_errors, @removed_errors, @new_violations, @removed_violations = runner.run
@new_errors, @removed_errors, @new_violations, @removed_violations, @new_configerrors, @removed_configerrors = runner.run
upload_report
rescue StandardError => e
warn("Running pmdtester failed, this message is mainly used to remind the maintainers of PMD.")
Expand All @@ -27,14 +27,17 @@ def upload_report
end

`tar -cf #{tar_filename} diff/`
report_url = `curl -u #{ENV['CHUNK_TOKEN']} -T #{tar_filename} chunk.io`
if $?.success?
@logger.info "Successfully uploaded #{tar_filename} to chunk.io"
report_url = `curl -u #{ENV['CHUNK_TOKEN']} -T #{tar_filename} chunk.io`
if $?.success?
@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("This changeset introduces #{@new_violations} new violations and #{@new_errors} new errors,\n" +
"removes #{@removed_violations} violations and #{@removed_errors} errors. [Full report](#{report_url.chomp}/diff/index.html)", sticky: true)
else
message("This changeset introduces #{@new_violations} new violations, #{@new_errors} new errors and " +
"#{@new_configerrors} new configuration errors,\n" +
"removes #{@removed_violations} violations, #{@removed_errors} errors and " +
"#{@removed_configerrors} configuration errors.\n" +
"[Full report](#{report_url.chomp}/diff/index.html)", sticky: true)
else
@logger.error "Error while uploading #{tar_filename} to chunk.io: #{report_url}"
warn("Uploading the diff report failed, this message is mainly used to remind the maintainers of PMD.")
end
Expand Down

0 comments on commit ecd60e8

Please sign in to comment.