Skip to content

Commit

Permalink
Merge pull request #555 from danger/sunshinejr-patch-1
Browse files Browse the repository at this point in the history
Don't print error for commit status when there was no error.
  • Loading branch information
orta committed Apr 3, 2018
2 parents cb2994f + 2a4ca06 commit 71aa70a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion source/runner/Executor.ts
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ export class Executor {
const dangerID = this.options.dangerID
const failed = fails.length > 0
const successPosting = await this.platform.updateStatus(!failed, messageForResults(results), this.ciSource.ciRunURL)
if (this.options.verbose) {
if (!successPosting && this.options.verbose) {
console.log("Could not add a commit status, the GitHub token for Danger does not have access rights.")
console.log("If the build fails, then danger will use a failing exit code.")
}
Expand Down

0 comments on commit 71aa70a

Please sign in to comment.