GitHub PullRequestReviewCommentEvent at 2014-01-20 23:11:09
Repository: logstash
Actor: jordansissel
Payload: {"comment":{"url":"https://api.github.com/repos/logstash/logstash/pulls/comments/9022013","id":9022013,"diff_hunk":"@@ -46,12 +57,11 @@ def run(queue)\n             queue << event\n           end\n         end\n+        stop = @restart != \"always\"","path":"lib/logstash/inputs/pipe.rb","position":31,"original_position":31,"commit_id":"020ed57fadf112dba24f5c6cc85d1d5524362030","original_commit_id":"020ed57fadf112dba24f5c6cc85d1d5524362030","user":{"login":"jordansissel","id":131818,"avatar_url":"https://gravatar.com/avatar/75fdd336ae5f818276f1cf79e0468b2d?d=https%3A%2F%2Fidenticons.github.com%2F983c25e4d13d98f06c505afe2a937b4d.png&r=x","gravatar_id":"75fdd336ae5f818276f1cf79e0468b2d","url":"https://api.github.com/users/jordansissel","html_url":"https://github.com/jordansissel","followers_url":"https://api.github.com/users/jordansissel/followers","following_url":"https://api.github.com/users/jordansissel/following{/other_user}","gists_url":"https://api.github.com/users/jordansissel/gists{/gist_id}","starred_url":"https://api.github.com/users/jordansissel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jordansissel/subscriptions","organizations_url":"https://api.github.com/users/jordansissel/orgs","repos_url":"https://api.github.com/users/jordansissel/repos","events_url":"https://api.github.com/users/jordansissel/events{/privacy}","received_events_url":"https://api.github.com/users/jordansissel/received_events","type":"User","site_admin":false},"body":"The double-negatives here make my brain trip up a bit; 'stop' being a negative and '!=' being negative, but we can leave the code as-is.","created_at":"2014-01-20T23:11:09Z","updated_at":"2014-01-20T23:11:09Z","html_url":"https://github.com/logstash/logstash/pull/942#discussion_r9022013","pull_request_url":"https://api.github.com/repos/logstash/logstash/pulls/942","_links":{"self":{"href":"https://api.github.com/repos/logstash/logstash/pulls/comments/9022013"},"html":{"href":"https://github.com/logstash/logstash/pull/942#discussion_r9022013"},"pull_request":{"href":"https://api.github.com/repos/logstash/logstash/pulls/942"}}}}