{"action":"created","issue":{"url":"https://api.github.com/repos/danger/danger-js/issues/756","repository_url":"https://api.github.com/repos/danger/danger-js","labels_url":"https://api.github.com/repos/danger/danger-js/issues/756/labels{/name}","comments_url":"https://api.github.com/repos/danger/danger-js/issues/756/comments","events_url":"https://api.github.com/repos/danger/danger-js/issues/756/events","html_url":"https://github.com/danger/danger-js/pull/756","id":377942895,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI4NzY0ODc5","number":756,"title":"Fix GitHub Checks API payload","user":{"login":"pveyes","id":1614415,"node_id":"MDQ6VXNlcjE2MTQ0MTU=","avatar_url":"https://avatars3.githubusercontent.com/u/1614415?v=4","gravatar_id":"","url":"https://api.github.com/users/pveyes","html_url":"https://github.com/pveyes","followers_url":"https://api.github.com/users/pveyes/followers","following_url":"https://api.github.com/users/pveyes/following{/other_user}","gists_url":"https://api.github.com/users/pveyes/gists{/gist_id}","starred_url":"https://api.github.com/users/pveyes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pveyes/subscriptions","organizations_url":"https://api.github.com/users/pveyes/orgs","repos_url":"https://api.github.com/users/pveyes/repos","events_url":"https://api.github.com/users/pveyes/events{/privacy}","received_events_url":"https://api.github.com/users/pveyes/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2018-11-06T17:00:22Z","updated_at":"2018-11-07T13:36:59Z","closed_at":"2018-11-07T13:36:32Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/danger/danger-js/pulls/756","html_url":"https://github.com/danger/danger-js/pull/756","diff_url":"https://github.com/danger/danger-js/pull/756.diff","patch_url":"https://github.com/danger/danger-js/pull/756.patch"},"body":"This fixes issue because of GitHub Checks API uses different fields (`filename` -> `path`, and `warning_level` -> `annotation_level`) https://developer.github.com/v3/checks/runs/#create-a-check-run\r\n\r\nI also separate how danger create annotations. Previously danger use 1 annotation for one file/line, but in our cases we found out that it's possible to have different `annotation_level` in the same file and same line (for example 2 different lint rules). With this change, every `fails`, `warnings`, `messages`, and `markdowns` will have its own annotation.\r\n\r\n~The last thing that I found out is that markdown is not supported in `annotation.message` so I just ignore `result.markdowns` (maybe we can put it inside `annotation.raw_message`?)~\r\n\r\nIf you want to test it you can change your `danger` dependency to `https://github.com/pveyes/danger-js.git#40f6b1acd165bd8953ab6862e1f9c260081eee10` "},"comment":{"url":"https://api.github.com/repos/danger/danger-js/issues/comments/436624759","html_url":"https://github.com/danger/danger-js/pull/756#issuecomment-436624759","issue_url":"https://api.github.com/repos/danger/danger-js/issues/756","id":436624759,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNjYyNDc1OQ==","user":{"login":"peril-staging[bot]","id":34651588,"node_id":"MDM6Qm90MzQ2NTE1ODg=","avatar_url":"https://avatars0.githubusercontent.com/in/7540?v=4","gravatar_id":"","url":"https://api.github.com/users/peril-staging%5Bbot%5D","html_url":"https://github.com/apps/peril-staging","followers_url":"https://api.github.com/users/peril-staging%5Bbot%5D/followers","following_url":"https://api.github.com/users/peril-staging%5Bbot%5D/following{/other_user}","gists_url":"https://api.github.com/users/peril-staging%5Bbot%5D/gists{/gist_id}","starred_url":"https://api.github.com/users/peril-staging%5Bbot%5D/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peril-staging%5Bbot%5D/subscriptions","organizations_url":"https://api.github.com/users/peril-staging%5Bbot%5D/orgs","repos_url":"https://api.github.com/users/peril-staging%5Bbot%5D/repos","events_url":"https://api.github.com/users/peril-staging%5Bbot%5D/events{/privacy}","received_events_url":"https://api.github.com/users/peril-staging%5Bbot%5D/received_events","type":"Bot","site_admin":false},"created_at":"2018-11-07T13:36:59Z","updated_at":"2018-11-07T13:36:59Z","author_association":"CONTRIBUTOR","body":"\n<!--\n  0 failure: \n  0 warning: \n  \n  1 markdown notices\n  DangerID: danger-id-peril;\n-->\n\n\n\n\n  Thanks for the PR @pveyes.\n\n  The Danger org conform to the [Moya Community Continuity Guidelines][moya_cc], which means\n  that we want to offer any contributor the ability to control their destiny.\n  \n  So, we've sent you an org invite - thanks pveyes :tada:\n  \n  [moya_cc]: https://github.com/Moya/contributors#readme\n  \n<p align=\"right\">\n  Generated by :no_entry_sign: <a href=\"http://github.com/danger/danger-js/\">dangerJS</a>\n</p>\n"}}
{"actor":{"display_login":"peril-staging"}}