GitHub PullRequestReviewCommentEvent at 2014-08-15 13:55:37
Repository: black-belt
Actor: Almad
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/apiaryio/black-belt/pulls/comments/16292973","id":16292973,"diff_hunk":"@@ -15,3 +18,19 @@ def test_repo_owner(self):\n \n     def test_repo_name(self):\n         assert_equals('apiary-test', self.parsed['name'])\n+\n+class TestGithubPullRequestParsing(object):","path":"test/test_github.py","position":null,"original_position":20,"commit_id":"91a6706c101b7c3cd16cb6f35f690efc9409ef65","original_commit_id":"498656b9ed803958e6450e3466d1bb391ba1ba41","user":{"login":"Almad","id":44565,"avatar_url":"https://avatars.githubusercontent.com/u/44565?v=2","gravatar_id":"f38b7d2954a2494564ba1a55a66fff83","url":"https://api.github.com/users/Almad","html_url":"https://github.com/Almad","followers_url":"https://api.github.com/users/Almad/followers","following_url":"https://api.github.com/users/Almad/following{/other_user}","gists_url":"https://api.github.com/users/Almad/gists{/gist_id}","starred_url":"https://api.github.com/users/Almad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Almad/subscriptions","organizations_url":"https://api.github.com/users/Almad/orgs","repos_url":"https://api.github.com/users/Almad/repos","events_url":"https://api.github.com/users/Almad/events{/privacy}","received_events_url":"https://api.github.com/users/Almad/received_events","type":"User","site_admin":false},"body":"Also, you don't need `py.test` for it, `nose` supports function-only tests as well — but I like the suite-wide shared constants and decoupling of preparation and tests/asserts that gets complicated with function-only tests. ","created_at":"2014-08-15T13:55:37Z","updated_at":"2014-08-15T13:55:37Z","html_url":"https://github.com/apiaryio/black-belt/pull/5#discussion_r16292973","pull_request_url":"https://api.github.com/repos/apiaryio/black-belt/pulls/5","_links":{"self":{"href":"https://api.github.com/repos/apiaryio/black-belt/pulls/comments/16292973"},"html":{"href":"https://github.com/apiaryio/black-belt/pull/5#discussion_r16292973"},"pull_request":{"href":"https://api.github.com/repos/apiaryio/black-belt/pulls/5"}}},"pull_request":{"url":"https://api.github.com/repos/apiaryio/black-belt/pulls/5","id":19843608,"html_url":"https://github.com/apiaryio/black-belt/pull/5","diff_url":"https://github.com/apiaryio/black-belt/pull/5.diff","patch_url":"https://github.com/apiaryio/black-belt/pull/5.patch","issue_url":"https://api.github.com/repos/apiaryio/black-belt/issues/5","number":5,"state":"closed","locked":false,"title":"bb gh merge","user":{"login":"Almad","id":44565,"avatar_url":"https://avatars.githubusercontent.com/u/44565?v=2","gravatar_id":"f38b7d2954a2494564ba1a55a66fff83","url":"https://api.github.com/users/Almad","html_url":"https://github.com/Almad","followers_url":"https://api.github.com/users/Almad/followers","following_url":"https://api.github.com/users/Almad/following{/other_user}","gists_url":"https://api.github.com/users/Almad/gists{/gist_id}","starred_url":"https://api.github.com/users/Almad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Almad/subscriptions","organizations_url":"https://api.github.com/users/Almad/orgs","repos_url":"https://api.github.com/users/Almad/repos","events_url":"https://api.github.com/users/Almad/events{/privacy}","received_events_url":"https://api.github.com/users/Almad/received_events","type":"User","site_admin":false},"body":"\n\nPull request for [bb gh merge](https://trello.com/c/pkU0gunr/1807-bb-gh-merge).\n\n    ","created_at":"2014-08-15T13:22:49Z","updated_at":"2014-08-15T13:55:37Z","closed_at":"2014-08-15T13:55:06Z","merged_at":"2014-08-15T13:55:06Z","merge_commit_sha":"917ea159e4805320b2f03d0d0280ec269dade8e7","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/apiaryio/black-belt/pulls/5/commits","review_comments_url":"https://api.github.com/repos/apiaryio/black-belt/pulls/5/comments","review_comment_url":"https://api.github.com/repos/apiaryio/black-belt/pulls/comments/{number}","comments_url":"https://api.github.com/repos/apiaryio/black-belt/issues/5/comments","statuses_url":"https://api.github.com/repos/apiaryio/black-belt/statuses/91a6706c101b7c3cd16cb6f35f690efc9409ef65","head":{"label":"apiaryio:almad/gh-merge","ref":"almad/gh-merge","sha":"91a6706c101b7c3cd16cb6f35f690efc9409ef65","user":{"login":"apiaryio","id":765943,"avatar_url":"https://avatars.githubusercontent.com/u/765943?v=2","gravatar_id":"c64b5c51ede2fbd878595bf27bc4d630","url":"https://api.github.com/users/apiaryio","html_url":"https://github.com/apiaryio","followers_url":"https://api.github.com/users/apiaryio/followers","following_url":"https://api.github.com/users/apiaryio/following{/other_user}","gists_url":"https://api.github.com/users/apiaryio/gists{/gist_id}","starred_url":"https://api.github.com/users/apiaryio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apiaryio/subscriptions","organizations_url":"https://api.github.com/users/apiaryio/orgs","repos_url":"https://api.github.com/users/apiaryio/repos","events_url":"https://api.github.com/users/apiaryio/events{/privacy}","received_events_url":"https://api.github.com/users/apiaryio/received_events","type":"Organization","site_admin":false},"repo":{"id":16481697,"name":"black-belt","full_name":"apiaryio/black-belt","owner":{"login":"apiaryio","id":765943,"avatar_url":"https://avatars.githubusercontent.com/u/765943?v=2","gravatar_id":"c64b5c51ede2fbd878595bf27bc4d630","url":"https://api.github.com/users/apiaryio","html_url":"https://github.com/apiaryio","followers_url":"https://api.github.com/users/apiaryio/followers","following_url":"https://api.github.com/users/apiaryio/following{/other_user}","gists_url":"https://api.github.com/users/apiaryio/gists{/gist_id}","starred_url":"https://api.github.com/users/apiaryio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apiaryio/subscriptions","organizations_url":"https://api.github.com/users/apiaryio/orgs","repos_url":"https://api.github.com/users/apiaryio/repos","events_url":"https://api.github.com/users/apiaryio/events{/privacy}","received_events_url":"https://api.github.com/users/apiaryio/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/apiaryio/black-belt","description":"Internal toolbelt on steroids","fork":false,"url":"https://api.github.com/repos/apiaryio/black-belt","forks_url":"https://api.github.com/repos/apiaryio/black-belt/forks","keys_url":"https://api.github.com/repos/apiaryio/black-belt/keys{/key_id}","collaborators_url":"https://api.github.com/repos/apiaryio/black-belt/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/apiaryio/black-belt/teams","hooks_url":"https://api.github.com/repos/apiaryio/black-belt/hooks","issue_events_url":"https://api.github.com/repos/apiaryio/black-belt/issues/events{/number}","events_url":"https://api.github.com/repos/apiaryio/black-belt/events","assignees_url":"https://api.github.com/repos/apiaryio/black-belt/assignees{/user}","branches_url":"https://api.github.com/repos/apiaryio/black-belt/branches{/branch}","tags_url":"https://api.github.com/repos/apiaryio/black-belt/tags","blobs_url":"https://api.github.com/repos/apiaryio/black-belt/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/apiaryio/black-belt/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/apiaryio/black-belt/git/refs{/sha}","trees_url":"https://api.github.com/repos/apiaryio/black-belt/git/trees{/sha}","statuses_url":"https://api.github.com/repos/apiaryio/black-belt/statuses/{sha}","languages_url":"https://api.github.com/repos/apiaryio/black-belt/languages","stargazers_url":"https://api.github.com/repos/apiaryio/black-belt/stargazers","contributors_url":"https://api.github.com/repos/apiaryio/black-belt/contributors","subscribers_url":"https://api.github.com/repos/apiaryio/black-belt/subscribers","subscription_url":"https://api.github.com/repos/apiaryio/black-belt/subscription","commits_url":"https://api.github.com/repos/apiaryio/black-belt/commits{/sha}","git_commits_url":"https://api.github.com/repos/apiaryio/black-belt/git/commits{/sha}","comments_url":"https://api.github.com/repos/apiaryio/black-belt/comments{/number}","issue_comment_url":"https://api.github.com/repos/apiaryio/black-belt/issues/comments/{number}","contents_url":"https://api.github.com/repos/apiaryio/black-belt/contents/{+path}","compare_url":"https://api.github.com/repos/apiaryio/black-belt/compare/{base}...{head}","merges_url":"https://api.github.com/repos/apiaryio/black-belt/merges","archive_url":"https://api.github.com/repos/apiaryio/black-belt/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/apiaryio/black-belt/downloads","issues_url":"https://api.github.com/repos/apiaryio/black-belt/issues{/number}","pulls_url":"https://api.github.com/repos/apiaryio/black-belt/pulls{/number}","milestones_url":"https://api.github.com/repos/apiaryio/black-belt/milestones{/number}","notifications_url":"https://api.github.com/repos/apiaryio/black-belt/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/apiaryio/black-belt/labels{/name}","releases_url":"https://api.github.com/repos/apiaryio/black-belt/releases{/id}","created_at":"2014-02-03T14:28:56Z","updated_at":"2014-08-08T13:43:01Z","pushed_at":"2014-08-15T13:55:08Z","git_url":"git://github.com/apiaryio/black-belt.git","ssh_url":"git@github.com:apiaryio/black-belt.git","clone_url":"https://github.com/apiaryio/black-belt.git","svn_url":"https://github.com/apiaryio/black-belt","homepage":null,"size":938,"stargazers_count":4,"watchers_count":4,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":4,"default_branch":"master"}},"base":{"label":"apiaryio:master","ref":"master","sha":"bf42a4dba2affd61850b67983043e431e811ea6b","user":{"login":"apiaryio","id":765943,"avatar_url":"https://avatars.githubusercontent.com/u/765943?v=2","gravatar_id":"c64b5c51ede2fbd878595bf27bc4d630","url":"https://api.github.com/users/apiaryio","html_url":"https://github.com/apiaryio","followers_url":"https://api.github.com/users/apiaryio/followers","following_url":"https://api.github.com/users/apiaryio/following{/other_user}","gists_url":"https://api.github.com/users/apiaryio/gists{/gist_id}","starred_url":"https://api.github.com/users/apiaryio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apiaryio/subscriptions","organizations_url":"https://api.github.com/users/apiaryio/orgs","repos_url":"https://api.github.com/users/apiaryio/repos","events_url":"https://api.github.com/users/apiaryio/events{/privacy}","received_events_url":"https://api.github.com/users/apiaryio/received_events","type":"Organization","site_admin":false},"repo":{"id":16481697,"name":"black-belt","full_name":"apiaryio/black-belt","owner":{"login":"apiaryio","id":765943,"avatar_url":"https://avatars.githubusercontent.com/u/765943?v=2","gravatar_id":"c64b5c51ede2fbd878595bf27bc4d630","url":"https://api.github.com/users/apiaryio","html_url":"https://github.com/apiaryio","followers_url":"https://api.github.com/users/apiaryio/followers","following_url":"https://api.github.com/users/apiaryio/following{/other_user}","gists_url":"https://api.github.com/users/apiaryio/gists{/gist_id}","starred_url":"https://api.github.com/users/apiaryio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apiaryio/subscriptions","organizations_url":"https://api.github.com/users/apiaryio/orgs","repos_url":"https://api.github.com/users/apiaryio/repos","events_url":"https://api.github.com/users/apiaryio/events{/privacy}","received_events_url":"https://api.github.com/users/apiaryio/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/apiaryio/black-belt","description":"Internal toolbelt on steroids","fork":false,"url":"https://api.github.com/repos/apiaryio/black-belt","forks_url":"https://api.github.com/repos/apiaryio/black-belt/forks","keys_url":"https://api.github.com/repos/apiaryio/black-belt/keys{/key_id}","collaborators_url":"https://api.github.com/repos/apiaryio/black-belt/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/apiaryio/black-belt/teams","hooks_url":"https://api.github.com/repos/apiaryio/black-belt/hooks","issue_events_url":"https://api.github.com/repos/apiaryio/black-belt/issues/events{/number}","events_url":"https://api.github.com/repos/apiaryio/black-belt/events","assignees_url":"https://api.github.com/repos/apiaryio/black-belt/assignees{/user}","branches_url":"https://api.github.com/repos/apiaryio/black-belt/branches{/branch}","tags_url":"https://api.github.com/repos/apiaryio/black-belt/tags","blobs_url":"https://api.github.com/repos/apiaryio/black-belt/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/apiaryio/black-belt/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/apiaryio/black-belt/git/refs{/sha}","trees_url":"https://api.github.com/repos/apiaryio/black-belt/git/trees{/sha}","statuses_url":"https://api.github.com/repos/apiaryio/black-belt/statuses/{sha}","languages_url":"https://api.github.com/repos/apiaryio/black-belt/languages","stargazers_url":"https://api.github.com/repos/apiaryio/black-belt/stargazers","contributors_url":"https://api.github.com/repos/apiaryio/black-belt/contributors","subscribers_url":"https://api.github.com/repos/apiaryio/black-belt/subscribers","subscription_url":"https://api.github.com/repos/apiaryio/black-belt/subscription","commits_url":"https://api.github.com/repos/apiaryio/black-belt/commits{/sha}","git_commits_url":"https://api.github.com/repos/apiaryio/black-belt/git/commits{/sha}","comments_url":"https://api.github.com/repos/apiaryio/black-belt/comments{/number}","issue_comment_url":"https://api.github.com/repos/apiaryio/black-belt/issues/comments/{number}","contents_url":"https://api.github.com/repos/apiaryio/black-belt/contents/{+path}","compare_url":"https://api.github.com/repos/apiaryio/black-belt/compare/{base}...{head}","merges_url":"https://api.github.com/repos/apiaryio/black-belt/merges","archive_url":"https://api.github.com/repos/apiaryio/black-belt/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/apiaryio/black-belt/downloads","issues_url":"https://api.github.com/repos/apiaryio/black-belt/issues{/number}","pulls_url":"https://api.github.com/repos/apiaryio/black-belt/pulls{/number}","milestones_url":"https://api.github.com/repos/apiaryio/black-belt/milestones{/number}","notifications_url":"https://api.github.com/repos/apiaryio/black-belt/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/apiaryio/black-belt/labels{/name}","releases_url":"https://api.github.com/repos/apiaryio/black-belt/releases{/id}","created_at":"2014-02-03T14:28:56Z","updated_at":"2014-08-08T13:43:01Z","pushed_at":"2014-08-15T13:55:08Z","git_url":"git://github.com/apiaryio/black-belt.git","ssh_url":"git@github.com:apiaryio/black-belt.git","clone_url":"https://github.com/apiaryio/black-belt.git","svn_url":"https://github.com/apiaryio/black-belt","homepage":null,"size":938,"stargazers_count":4,"watchers_count":4,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":4,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/apiaryio/black-belt/pulls/5"},"html":{"href":"https://github.com/apiaryio/black-belt/pull/5"},"issue":{"href":"https://api.github.com/repos/apiaryio/black-belt/issues/5"},"comments":{"href":"https://api.github.com/repos/apiaryio/black-belt/issues/5/comments"},"review_comments":{"href":"https://api.github.com/repos/apiaryio/black-belt/pulls/5/comments"},"review_comment":{"href":"https://api.github.com/repos/apiaryio/black-belt/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/apiaryio/black-belt/pulls/5/commits"},"statuses":{"href":"https://api.github.com/repos/apiaryio/black-belt/statuses/91a6706c101b7c3cd16cb6f35f690efc9409ef65"}}}}