GitHub PullRequestReviewCommentEvent at 2014-07-21 07:21:18
Repository: hackernews-scraper
Actor: NiGhTTraX
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/comments/15156977","id":15156977,"diff_hunk":"@@ -1,4 +1,4 @@\n-__version__ = \"1.0.1\"\n+__version__ = \"1.0.2\"","path":"hackernews_scraper/__init__.py","position":2,"original_position":2,"commit_id":"09d13d228d2f640475e79f4461c50e5b372e53bd","original_commit_id":"09d13d228d2f640475e79f4461c50e5b372e53bd","user":{"login":"NiGhTTraX","id":485061,"avatar_url":"https://avatars.githubusercontent.com/u/485061?","gravatar_id":"96e218c50bc2c9c0afb19ced54b87d16","url":"https://api.github.com/users/NiGhTTraX","html_url":"https://github.com/NiGhTTraX","followers_url":"https://api.github.com/users/NiGhTTraX/followers","following_url":"https://api.github.com/users/NiGhTTraX/following{/other_user}","gists_url":"https://api.github.com/users/NiGhTTraX/gists{/gist_id}","starred_url":"https://api.github.com/users/NiGhTTraX/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NiGhTTraX/subscriptions","organizations_url":"https://api.github.com/users/NiGhTTraX/orgs","repos_url":"https://api.github.com/users/NiGhTTraX/repos","events_url":"https://api.github.com/users/NiGhTTraX/events{/privacy}","received_events_url":"https://api.github.com/users/NiGhTTraX/received_events","type":"User","site_admin":false},"body":"Don't bump the version. We're just updating tests, there's no visible change to the users i.e. they don't need to download the latest version.","created_at":"2014-07-21T07:21:18Z","updated_at":"2014-07-21T07:21:18Z","html_url":"https://github.com/NiGhTTraX/hackernews-scraper/pull/14#discussion_r15156977","pull_request_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/14","_links":{"self":{"href":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/comments/15156977"},"html":{"href":"https://github.com/NiGhTTraX/hackernews-scraper/pull/14#discussion_r15156977"},"pull_request":{"href":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/14"}}},"pull_request":{"url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/14","id":18626999,"html_url":"https://github.com/NiGhTTraX/hackernews-scraper/pull/14","diff_url":"https://github.com/NiGhTTraX/hackernews-scraper/pull/14.diff","patch_url":"https://github.com/NiGhTTraX/hackernews-scraper/pull/14.patch","issue_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/issues/14","number":14,"state":"open","title":"PR for factories","user":{"login":"palcu","id":772220,"avatar_url":"https://avatars.githubusercontent.com/u/772220?","gravatar_id":"1bcbc663a5a9e0c1b1766a45dd90a236","url":"https://api.github.com/users/palcu","html_url":"https://github.com/palcu","followers_url":"https://api.github.com/users/palcu/followers","following_url":"https://api.github.com/users/palcu/following{/other_user}","gists_url":"https://api.github.com/users/palcu/gists{/gist_id}","starred_url":"https://api.github.com/users/palcu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/palcu/subscriptions","organizations_url":"https://api.github.com/users/palcu/orgs","repos_url":"https://api.github.com/users/palcu/repos","events_url":"https://api.github.com/users/palcu/events{/privacy}","received_events_url":"https://api.github.com/users/palcu/received_events","type":"User","site_admin":false},"body":"Since we are using `CommentFactory` from `factories.py` in production, I made sure that it returns random correct data. Tests are all passing.","created_at":"2014-07-20T05:42:01Z","updated_at":"2014-07-21T07:21:18Z","closed_at":null,"merged_at":null,"merge_commit_sha":"108f8da74f4db4a1668b64c6c1839e629400107f","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/14/commits","review_comments_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/14/comments","review_comment_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/comments/{number}","comments_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/issues/14/comments","statuses_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/statuses/09d13d228d2f640475e79f4461c50e5b372e53bd","head":{"label":"palcu:master","ref":"master","sha":"09d13d228d2f640475e79f4461c50e5b372e53bd","user":{"login":"palcu","id":772220,"avatar_url":"https://avatars.githubusercontent.com/u/772220?","gravatar_id":"1bcbc663a5a9e0c1b1766a45dd90a236","url":"https://api.github.com/users/palcu","html_url":"https://github.com/palcu","followers_url":"https://api.github.com/users/palcu/followers","following_url":"https://api.github.com/users/palcu/following{/other_user}","gists_url":"https://api.github.com/users/palcu/gists{/gist_id}","starred_url":"https://api.github.com/users/palcu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/palcu/subscriptions","organizations_url":"https://api.github.com/users/palcu/orgs","repos_url":"https://api.github.com/users/palcu/repos","events_url":"https://api.github.com/users/palcu/events{/privacy}","received_events_url":"https://api.github.com/users/palcu/received_events","type":"User","site_admin":false},"repo":{"id":21691353,"name":"hackernews-scraper","full_name":"palcu/hackernews-scraper","owner":{"login":"palcu","id":772220,"avatar_url":"https://avatars.githubusercontent.com/u/772220?","gravatar_id":"1bcbc663a5a9e0c1b1766a45dd90a236","url":"https://api.github.com/users/palcu","html_url":"https://github.com/palcu","followers_url":"https://api.github.com/users/palcu/followers","following_url":"https://api.github.com/users/palcu/following{/other_user}","gists_url":"https://api.github.com/users/palcu/gists{/gist_id}","starred_url":"https://api.github.com/users/palcu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/palcu/subscriptions","organizations_url":"https://api.github.com/users/palcu/orgs","repos_url":"https://api.github.com/users/palcu/repos","events_url":"https://api.github.com/users/palcu/events{/privacy}","received_events_url":"https://api.github.com/users/palcu/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/palcu/hackernews-scraper","description":"Scrape hacker news comments and posts","fork":true,"url":"https://api.github.com/repos/palcu/hackernews-scraper","forks_url":"https://api.github.com/repos/palcu/hackernews-scraper/forks","keys_url":"https://api.github.com/repos/palcu/hackernews-scraper/keys{/key_id}","collaborators_url":"https://api.github.com/repos/palcu/hackernews-scraper/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/palcu/hackernews-scraper/teams","hooks_url":"https://api.github.com/repos/palcu/hackernews-scraper/hooks","issue_events_url":"https://api.github.com/repos/palcu/hackernews-scraper/issues/events{/number}","events_url":"https://api.github.com/repos/palcu/hackernews-scraper/events","assignees_url":"https://api.github.com/repos/palcu/hackernews-scraper/assignees{/user}","branches_url":"https://api.github.com/repos/palcu/hackernews-scraper/branches{/branch}","tags_url":"https://api.github.com/repos/palcu/hackernews-scraper/tags","blobs_url":"https://api.github.com/repos/palcu/hackernews-scraper/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/palcu/hackernews-scraper/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/palcu/hackernews-scraper/git/refs{/sha}","trees_url":"https://api.github.com/repos/palcu/hackernews-scraper/git/trees{/sha}","statuses_url":"https://api.github.com/repos/palcu/hackernews-scraper/statuses/{sha}","languages_url":"https://api.github.com/repos/palcu/hackernews-scraper/languages","stargazers_url":"https://api.github.com/repos/palcu/hackernews-scraper/stargazers","contributors_url":"https://api.github.com/repos/palcu/hackernews-scraper/contributors","subscribers_url":"https://api.github.com/repos/palcu/hackernews-scraper/subscribers","subscription_url":"https://api.github.com/repos/palcu/hackernews-scraper/subscription","commits_url":"https://api.github.com/repos/palcu/hackernews-scraper/commits{/sha}","git_commits_url":"https://api.github.com/repos/palcu/hackernews-scraper/git/commits{/sha}","comments_url":"https://api.github.com/repos/palcu/hackernews-scraper/comments{/number}","issue_comment_url":"https://api.github.com/repos/palcu/hackernews-scraper/issues/comments/{number}","contents_url":"https://api.github.com/repos/palcu/hackernews-scraper/contents/{+path}","compare_url":"https://api.github.com/repos/palcu/hackernews-scraper/compare/{base}...{head}","merges_url":"https://api.github.com/repos/palcu/hackernews-scraper/merges","archive_url":"https://api.github.com/repos/palcu/hackernews-scraper/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/palcu/hackernews-scraper/downloads","issues_url":"https://api.github.com/repos/palcu/hackernews-scraper/issues{/number}","pulls_url":"https://api.github.com/repos/palcu/hackernews-scraper/pulls{/number}","milestones_url":"https://api.github.com/repos/palcu/hackernews-scraper/milestones{/number}","notifications_url":"https://api.github.com/repos/palcu/hackernews-scraper/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/palcu/hackernews-scraper/labels{/name}","releases_url":"https://api.github.com/repos/palcu/hackernews-scraper/releases{/id}","created_at":"2014-07-10T11:10:41Z","updated_at":"2014-07-11T11:18:22Z","pushed_at":"2014-07-21T06:42:07Z","git_url":"git://github.com/palcu/hackernews-scraper.git","ssh_url":"git@github.com:palcu/hackernews-scraper.git","clone_url":"https://github.com/palcu/hackernews-scraper.git","svn_url":"https://github.com/palcu/hackernews-scraper","homepage":null,"size":315,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"NiGhTTraX:master","ref":"master","sha":"f757f9dbb42bfe06f08e8799156d926435480aff","user":{"login":"NiGhTTraX","id":485061,"avatar_url":"https://avatars.githubusercontent.com/u/485061?","gravatar_id":"96e218c50bc2c9c0afb19ced54b87d16","url":"https://api.github.com/users/NiGhTTraX","html_url":"https://github.com/NiGhTTraX","followers_url":"https://api.github.com/users/NiGhTTraX/followers","following_url":"https://api.github.com/users/NiGhTTraX/following{/other_user}","gists_url":"https://api.github.com/users/NiGhTTraX/gists{/gist_id}","starred_url":"https://api.github.com/users/NiGhTTraX/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NiGhTTraX/subscriptions","organizations_url":"https://api.github.com/users/NiGhTTraX/orgs","repos_url":"https://api.github.com/users/NiGhTTraX/repos","events_url":"https://api.github.com/users/NiGhTTraX/events{/privacy}","received_events_url":"https://api.github.com/users/NiGhTTraX/received_events","type":"User","site_admin":false},"repo":{"id":17705809,"name":"hackernews-scraper","full_name":"NiGhTTraX/hackernews-scraper","owner":{"login":"NiGhTTraX","id":485061,"avatar_url":"https://avatars.githubusercontent.com/u/485061?","gravatar_id":"96e218c50bc2c9c0afb19ced54b87d16","url":"https://api.github.com/users/NiGhTTraX","html_url":"https://github.com/NiGhTTraX","followers_url":"https://api.github.com/users/NiGhTTraX/followers","following_url":"https://api.github.com/users/NiGhTTraX/following{/other_user}","gists_url":"https://api.github.com/users/NiGhTTraX/gists{/gist_id}","starred_url":"https://api.github.com/users/NiGhTTraX/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NiGhTTraX/subscriptions","organizations_url":"https://api.github.com/users/NiGhTTraX/orgs","repos_url":"https://api.github.com/users/NiGhTTraX/repos","events_url":"https://api.github.com/users/NiGhTTraX/events{/privacy}","received_events_url":"https://api.github.com/users/NiGhTTraX/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/NiGhTTraX/hackernews-scraper","description":"Scrape hacker news comments and posts","fork":false,"url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper","forks_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/forks","keys_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/keys{/key_id}","collaborators_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/teams","hooks_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/hooks","issue_events_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/issues/events{/number}","events_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/events","assignees_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/assignees{/user}","branches_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/branches{/branch}","tags_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/tags","blobs_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/git/refs{/sha}","trees_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/git/trees{/sha}","statuses_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/statuses/{sha}","languages_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/languages","stargazers_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/stargazers","contributors_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/contributors","subscribers_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/subscribers","subscription_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/subscription","commits_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/commits{/sha}","git_commits_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/git/commits{/sha}","comments_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/comments{/number}","issue_comment_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/issues/comments/{number}","contents_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/contents/{+path}","compare_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/compare/{base}...{head}","merges_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/merges","archive_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/downloads","issues_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/issues{/number}","pulls_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls{/number}","milestones_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/milestones{/number}","notifications_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/labels{/name}","releases_url":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/releases{/id}","created_at":"2014-03-13T10:14:15Z","updated_at":"2014-06-27T09:16:43Z","pushed_at":"2014-07-11T13:06:40Z","git_url":"git://github.com/NiGhTTraX/hackernews-scraper.git","ssh_url":"git@github.com:NiGhTTraX/hackernews-scraper.git","clone_url":"https://github.com/NiGhTTraX/hackernews-scraper.git","svn_url":"https://github.com/NiGhTTraX/hackernews-scraper","homepage":null,"size":524,"stargazers_count":4,"watchers_count":4,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"forks_count":1,"mirror_url":null,"open_issues_count":2,"forks":1,"open_issues":2,"watchers":4,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/14"},"html":{"href":"https://github.com/NiGhTTraX/hackernews-scraper/pull/14"},"issue":{"href":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/issues/14"},"comments":{"href":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/issues/14/comments"},"review_comments":{"href":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/14/comments"},"review_comment":{"href":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/pulls/14/commits"},"statuses":{"href":"https://api.github.com/repos/NiGhTTraX/hackernews-scraper/statuses/09d13d228d2f640475e79f4461c50e5b372e53bd"}}}}