Post status to github rather than breaking the build

[shilman]

why

Currently when there are any fail messages, Danger breaks the build:

danger-js/source/runner/Executor.ts

Line 156 in a45ad2f

process.exitCode = 1

If the token has PR access, it should post a build status instead rather than completely breaking the build.

what

Try to post the status, set exitCode to 1 if this fails

[macklinu]

Status posting has been shipped in danger@0.21.0 - @shilman, please update and let us know what you think!

Here's an example from a private repo at work where Danger found one warning and no failures, which marks the PR with a "success" status and the there-are-fixable-things message. 🙂

Let's keep this issue open to add some more tests around this feature. 👍

[shilman]

That was fast—thanks for the fast turnaround! And it works like a charm: storybookjs/storybook#1251