payload: {"action":"created","issue":{"url":"https://api.github.com/repos/palantir/policy-bot/issues/63","repository_url":"https://api.github.com/repos/palantir/policy-bot","labels_url":"https://api.github.com/repos/palantir/policy-bot/issues/63/labels{/name}","comments_url":"https://api.github.com/repos/palantir/policy-bot/issues/63/comments","events_url":"https://api.github.com/repos/palantir/policy-bot/issues/63/events","html_url":"https://github.com/palantir/policy-bot/issues/63","id":433218693,"node_id":"MDU6SXNzdWU0MzMyMTg2OTM=","number":63,"title":"Contributors ignored during a single evaluation","user":{"login":"iamdanfox","id":3473798,"node_id":"MDQ6VXNlcjM0NzM3OTg=","avatar_url":"https://avatars1.githubusercontent.com/u/3473798?v=4","gravatar_id":"","url":"https://api.github.com/users/iamdanfox","html_url":"https://github.com/iamdanfox","followers_url":"https://api.github.com/users/iamdanfox/followers","following_url":"https://api.github.com/users/iamdanfox/following{/other_user}","gists_url":"https://api.github.com/users/iamdanfox/gists{/gist_id}","starred_url":"https://api.github.com/users/iamdanfox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iamdanfox/subscriptions","organizations_url":"https://api.github.com/users/iamdanfox/orgs","repos_url":"https://api.github.com/users/iamdanfox/repos","events_url":"https://api.github.com/users/iamdanfox/events{/privacy}","received_events_url":"https://api.github.com/users/iamdanfox/received_events","type":"User","site_admin":false},"labels":[{"id":1079862592,"node_id":"MDU6TGFiZWwxMDc5ODYyNTky","url":"https://api.github.com/repos/palantir/policy-bot/labels/bug","name":"bug","color":"d73a4a","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2019-04-15T10:57:45Z","updated_at":"2019-04-18T21:09:29Z","closed_at":null,"author_association":"NONE","body":"**Edit:** see [comment](https://github.com/palantir/policy-bot/issues/63#issuecomment-483497069) for a summary of the underlying bug.\r\n\r\n----\r\n\r\nI left a comment on a single line of this PR (https://github.com/palantir/gradle-conjure/pull/136), but it seems it caused the entire thing to merge.\r\n\r\n<img width=\"863\" alt=\"Screenshot 2019-04-15 at 13 15 10\" src=\"https://user-images.githubusercontent.com/3473798/56131854-87c81480-5f80-11e9-8438-ceefd64be029.png\">\r\n\r\nInterestingly, the UI seems to still say the PR has not yet been approved, but the status check went green so bulldozer mnerged it.\r\n\r\n<img width=\"800\" alt=\"Screenshot 2019-04-15 at 11 57 15\" src=\"https://user-images.githubusercontent.com/3473798/56127703-aa086500-5f75-11e9-9767-6111c1c92a6a.png\">\r\n"},"comment":{"url":"https://api.github.com/repos/palantir/policy-bot/issues/comments/484690646","html_url":"https://github.com/palantir/policy-bot/issues/63#issuecomment-484690646","issue_url":"https://api.github.com/repos/palantir/policy-bot/issues/63","id":484690646,"node_id":"MDEyOklzc3VlQ29tbWVudDQ4NDY5MDY0Ng==","user":{"login":"bluekeyes","id":1745813,"node_id":"MDQ6VXNlcjE3NDU4MTM=","avatar_url":"https://avatars0.githubusercontent.com/u/1745813?v=4","gravatar_id":"","url":"https://api.github.com/users/bluekeyes","html_url":"https://github.com/bluekeyes","followers_url":"https://api.github.com/users/bluekeyes/followers","following_url":"https://api.github.com/users/bluekeyes/following{/other_user}","gists_url":"https://api.github.com/users/bluekeyes/gists{/gist_id}","starred_url":"https://api.github.com/users/bluekeyes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bluekeyes/subscriptions","organizations_url":"https://api.github.com/users/bluekeyes/orgs","repos_url":"https://api.github.com/users/bluekeyes/repos","events_url":"https://api.github.com/users/bluekeyes/events{/privacy}","received_events_url":"https://api.github.com/users/bluekeyes/received_events","type":"User","site_admin":false},"created_at":"2019-04-18T21:09:29Z","updated_at":"2019-04-18T21:09:29Z","author_association":"MEMBER","body":"I believe the root cause of this problem is that the new commit loading in 1.7.0 uses the commit count reported as part of the pull request object to know how many commits to list, but `pull_request_review` events (and only these events as far as I can tell) include a partial pull request object that is inexplicably missing this field (despite it appearing in the documentation). That leads us to list zero commits and lose all contributor information.\r\n\r\nIf the policy used the `invalidate_on_push` option, this would have caused a crash instead, which is the other problem we've been seeing with version 1.7.0."}}
other: {"actor":{"display_login":"bluekeyes"}}