GitHub PullRequestReviewCommentEvent at 2014-06-29 15:07:16
Repository: BuildCraft
Actor: SpaceToad
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/comments/14329774","id":14329774,"diff_hunk":"@@ -589,7 +589,7 @@ public ForgeDirection getOpenOrientation() {\n \t\tForgeDirection targetOrientation = ForgeDirection.UNKNOWN;\n \n \t\tfor (ForgeDirection o : ForgeDirection.VALID_DIRECTIONS) {\n-\t\t\tif (container.isPipeConnected(o)) {\n+\t\t\tif (container.pipeConnectionsBuffer[o.ordinal()]) {","path":"common/buildcraft/transport/Pipe.java","position":14,"original_position":14,"commit_id":"69aa89e7aff3abb02baa0764167741d00bccecac","original_commit_id":"69aa89e7aff3abb02baa0764167741d00bccecac","user":{"login":"SpaceToad","id":1412063,"avatar_url":"https://avatars.githubusercontent.com/u/1412063?","gravatar_id":"cce5980f6f466eae3bb4e122e81e7ce9","url":"https://api.github.com/users/SpaceToad","html_url":"https://github.com/SpaceToad","followers_url":"https://api.github.com/users/SpaceToad/followers","following_url":"https://api.github.com/users/SpaceToad/following{/other_user}","gists_url":"https://api.github.com/users/SpaceToad/gists{/gist_id}","starred_url":"https://api.github.com/users/SpaceToad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SpaceToad/subscriptions","organizations_url":"https://api.github.com/users/SpaceToad/orgs","repos_url":"https://api.github.com/users/SpaceToad/repos","events_url":"https://api.github.com/users/SpaceToad/events{/privacy}","received_events_url":"https://api.github.com/users/SpaceToad/received_events","type":"User","site_admin":false},"body":"Same as before - why is this change needed? That doesn't seem related to the goal of this modification.","created_at":"2014-06-29T15:07:16Z","updated_at":"2014-06-29T15:07:16Z","html_url":"https://github.com/BuildCraft/BuildCraft/pull/1787#discussion_r14329774","pull_request_url":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/1787","_links":{"self":{"href":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/comments/14329774"},"html":{"href":"https://github.com/BuildCraft/BuildCraft/pull/1787#discussion_r14329774"},"pull_request":{"href":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/1787"}}},"pull_request":{"url":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/1787","id":15744670,"html_url":"https://github.com/BuildCraft/BuildCraft/pull/1787","diff_url":"https://github.com/BuildCraft/BuildCraft/pull/1787.diff","patch_url":"https://github.com/BuildCraft/BuildCraft/pull/1787.patch","issue_url":"https://api.github.com/repos/BuildCraft/BuildCraft/issues/1787","number":1787,"state":"open","title":"Stripes pipe API","user":{"login":"anti344","id":3393390,"avatar_url":"https://avatars.githubusercontent.com/u/3393390?","gravatar_id":"b9dc629e7c6ec25b976f24c4bfd40246","url":"https://api.github.com/users/anti344","html_url":"https://github.com/anti344","followers_url":"https://api.github.com/users/anti344/followers","following_url":"https://api.github.com/users/anti344/following{/other_user}","gists_url":"https://api.github.com/users/anti344/gists{/gist_id}","starred_url":"https://api.github.com/users/anti344/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anti344/subscriptions","organizations_url":"https://api.github.com/users/anti344/orgs","repos_url":"https://api.github.com/users/anti344/repos","events_url":"https://api.github.com/users/anti344/events{/privacy}","received_events_url":"https://api.github.com/users/anti344/received_events","type":"User","site_admin":false},"body":"Sorry about my Location class, but it SOO USEFUL.","created_at":"2014-05-11T17:56:46Z","updated_at":"2014-06-29T15:07:16Z","closed_at":null,"merged_at":null,"merge_commit_sha":"3ddbcfe3aed77de7c9bfe5555835a3ccda6832a4","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/1787/commits","review_comments_url":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/1787/comments","review_comment_url":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/comments/{number}","comments_url":"https://api.github.com/repos/BuildCraft/BuildCraft/issues/1787/comments","statuses_url":"https://api.github.com/repos/BuildCraft/BuildCraft/statuses/69aa89e7aff3abb02baa0764167741d00bccecac","head":{"label":"anti344:6.1.x","ref":"6.1.x","sha":"69aa89e7aff3abb02baa0764167741d00bccecac","user":{"login":"anti344","id":3393390,"avatar_url":"https://avatars.githubusercontent.com/u/3393390?","gravatar_id":"b9dc629e7c6ec25b976f24c4bfd40246","url":"https://api.github.com/users/anti344","html_url":"https://github.com/anti344","followers_url":"https://api.github.com/users/anti344/followers","following_url":"https://api.github.com/users/anti344/following{/other_user}","gists_url":"https://api.github.com/users/anti344/gists{/gist_id}","starred_url":"https://api.github.com/users/anti344/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anti344/subscriptions","organizations_url":"https://api.github.com/users/anti344/orgs","repos_url":"https://api.github.com/users/anti344/repos","events_url":"https://api.github.com/users/anti344/events{/privacy}","received_events_url":"https://api.github.com/users/anti344/received_events","type":"User","site_admin":false},"repo":{"id":19639055,"name":"BuildCraft","full_name":"anti344/BuildCraft","owner":{"login":"anti344","id":3393390,"avatar_url":"https://avatars.githubusercontent.com/u/3393390?","gravatar_id":"b9dc629e7c6ec25b976f24c4bfd40246","url":"https://api.github.com/users/anti344","html_url":"https://github.com/anti344","followers_url":"https://api.github.com/users/anti344/followers","following_url":"https://api.github.com/users/anti344/following{/other_user}","gists_url":"https://api.github.com/users/anti344/gists{/gist_id}","starred_url":"https://api.github.com/users/anti344/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anti344/subscriptions","organizations_url":"https://api.github.com/users/anti344/orgs","repos_url":"https://api.github.com/users/anti344/repos","events_url":"https://api.github.com/users/anti344/events{/privacy}","received_events_url":"https://api.github.com/users/anti344/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/anti344/BuildCraft","description":"BuildCraft","fork":true,"url":"https://api.github.com/repos/anti344/BuildCraft","forks_url":"https://api.github.com/repos/anti344/BuildCraft/forks","keys_url":"https://api.github.com/repos/anti344/BuildCraft/keys{/key_id}","collaborators_url":"https://api.github.com/repos/anti344/BuildCraft/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/anti344/BuildCraft/teams","hooks_url":"https://api.github.com/repos/anti344/BuildCraft/hooks","issue_events_url":"https://api.github.com/repos/anti344/BuildCraft/issues/events{/number}","events_url":"https://api.github.com/repos/anti344/BuildCraft/events","assignees_url":"https://api.github.com/repos/anti344/BuildCraft/assignees{/user}","branches_url":"https://api.github.com/repos/anti344/BuildCraft/branches{/branch}","tags_url":"https://api.github.com/repos/anti344/BuildCraft/tags","blobs_url":"https://api.github.com/repos/anti344/BuildCraft/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/anti344/BuildCraft/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/anti344/BuildCraft/git/refs{/sha}","trees_url":"https://api.github.com/repos/anti344/BuildCraft/git/trees{/sha}","statuses_url":"https://api.github.com/repos/anti344/BuildCraft/statuses/{sha}","languages_url":"https://api.github.com/repos/anti344/BuildCraft/languages","stargazers_url":"https://api.github.com/repos/anti344/BuildCraft/stargazers","contributors_url":"https://api.github.com/repos/anti344/BuildCraft/contributors","subscribers_url":"https://api.github.com/repos/anti344/BuildCraft/subscribers","subscription_url":"https://api.github.com/repos/anti344/BuildCraft/subscription","commits_url":"https://api.github.com/repos/anti344/BuildCraft/commits{/sha}","git_commits_url":"https://api.github.com/repos/anti344/BuildCraft/git/commits{/sha}","comments_url":"https://api.github.com/repos/anti344/BuildCraft/comments{/number}","issue_comment_url":"https://api.github.com/repos/anti344/BuildCraft/issues/comments/{number}","contents_url":"https://api.github.com/repos/anti344/BuildCraft/contents/{+path}","compare_url":"https://api.github.com/repos/anti344/BuildCraft/compare/{base}...{head}","merges_url":"https://api.github.com/repos/anti344/BuildCraft/merges","archive_url":"https://api.github.com/repos/anti344/BuildCraft/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/anti344/BuildCraft/downloads","issues_url":"https://api.github.com/repos/anti344/BuildCraft/issues{/number}","pulls_url":"https://api.github.com/repos/anti344/BuildCraft/pulls{/number}","milestones_url":"https://api.github.com/repos/anti344/BuildCraft/milestones{/number}","notifications_url":"https://api.github.com/repos/anti344/BuildCraft/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/anti344/BuildCraft/labels{/name}","releases_url":"https://api.github.com/repos/anti344/BuildCraft/releases{/id}","created_at":"2014-05-10T10:42:35Z","updated_at":"2014-06-18T21:58:07Z","pushed_at":"2014-06-18T21:58:07Z","git_url":"git://github.com/anti344/BuildCraft.git","ssh_url":"git@github.com:anti344/BuildCraft.git","clone_url":"https://github.com/anti344/BuildCraft.git","svn_url":"https://github.com/anti344/BuildCraft","homepage":"http://www.mod-buildcraft.com","size":11844,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"6.1.x"}},"base":{"label":"BuildCraft:6.1.x","ref":"6.1.x","sha":"bcc0f37e82d0058f00b380a5da053ade050b9ea7","user":{"login":"BuildCraft","id":3163754,"avatar_url":"https://avatars.githubusercontent.com/u/3163754?","gravatar_id":"8e499214d81ac3a09a70a37065099f4c","url":"https://api.github.com/users/BuildCraft","html_url":"https://github.com/BuildCraft","followers_url":"https://api.github.com/users/BuildCraft/followers","following_url":"https://api.github.com/users/BuildCraft/following{/other_user}","gists_url":"https://api.github.com/users/BuildCraft/gists{/gist_id}","starred_url":"https://api.github.com/users/BuildCraft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BuildCraft/subscriptions","organizations_url":"https://api.github.com/users/BuildCraft/orgs","repos_url":"https://api.github.com/users/BuildCraft/repos","events_url":"https://api.github.com/users/BuildCraft/events{/privacy}","received_events_url":"https://api.github.com/users/BuildCraft/received_events","type":"Organization","site_admin":false},"repo":{"id":4039952,"name":"BuildCraft","full_name":"BuildCraft/BuildCraft","owner":{"login":"BuildCraft","id":3163754,"avatar_url":"https://avatars.githubusercontent.com/u/3163754?","gravatar_id":"8e499214d81ac3a09a70a37065099f4c","url":"https://api.github.com/users/BuildCraft","html_url":"https://github.com/BuildCraft","followers_url":"https://api.github.com/users/BuildCraft/followers","following_url":"https://api.github.com/users/BuildCraft/following{/other_user}","gists_url":"https://api.github.com/users/BuildCraft/gists{/gist_id}","starred_url":"https://api.github.com/users/BuildCraft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BuildCraft/subscriptions","organizations_url":"https://api.github.com/users/BuildCraft/orgs","repos_url":"https://api.github.com/users/BuildCraft/repos","events_url":"https://api.github.com/users/BuildCraft/events{/privacy}","received_events_url":"https://api.github.com/users/BuildCraft/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/BuildCraft/BuildCraft","description":"BuildCraft","fork":false,"url":"https://api.github.com/repos/BuildCraft/BuildCraft","forks_url":"https://api.github.com/repos/BuildCraft/BuildCraft/forks","keys_url":"https://api.github.com/repos/BuildCraft/BuildCraft/keys{/key_id}","collaborators_url":"https://api.github.com/repos/BuildCraft/BuildCraft/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/BuildCraft/BuildCraft/teams","hooks_url":"https://api.github.com/repos/BuildCraft/BuildCraft/hooks","issue_events_url":"https://api.github.com/repos/BuildCraft/BuildCraft/issues/events{/number}","events_url":"https://api.github.com/repos/BuildCraft/BuildCraft/events","assignees_url":"https://api.github.com/repos/BuildCraft/BuildCraft/assignees{/user}","branches_url":"https://api.github.com/repos/BuildCraft/BuildCraft/branches{/branch}","tags_url":"https://api.github.com/repos/BuildCraft/BuildCraft/tags","blobs_url":"https://api.github.com/repos/BuildCraft/BuildCraft/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/BuildCraft/BuildCraft/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/BuildCraft/BuildCraft/git/refs{/sha}","trees_url":"https://api.github.com/repos/BuildCraft/BuildCraft/git/trees{/sha}","statuses_url":"https://api.github.com/repos/BuildCraft/BuildCraft/statuses/{sha}","languages_url":"https://api.github.com/repos/BuildCraft/BuildCraft/languages","stargazers_url":"https://api.github.com/repos/BuildCraft/BuildCraft/stargazers","contributors_url":"https://api.github.com/repos/BuildCraft/BuildCraft/contributors","subscribers_url":"https://api.github.com/repos/BuildCraft/BuildCraft/subscribers","subscription_url":"https://api.github.com/repos/BuildCraft/BuildCraft/subscription","commits_url":"https://api.github.com/repos/BuildCraft/BuildCraft/commits{/sha}","git_commits_url":"https://api.github.com/repos/BuildCraft/BuildCraft/git/commits{/sha}","comments_url":"https://api.github.com/repos/BuildCraft/BuildCraft/comments{/number}","issue_comment_url":"https://api.github.com/repos/BuildCraft/BuildCraft/issues/comments/{number}","contents_url":"https://api.github.com/repos/BuildCraft/BuildCraft/contents/{+path}","compare_url":"https://api.github.com/repos/BuildCraft/BuildCraft/compare/{base}...{head}","merges_url":"https://api.github.com/repos/BuildCraft/BuildCraft/merges","archive_url":"https://api.github.com/repos/BuildCraft/BuildCraft/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/BuildCraft/BuildCraft/downloads","issues_url":"https://api.github.com/repos/BuildCraft/BuildCraft/issues{/number}","pulls_url":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls{/number}","milestones_url":"https://api.github.com/repos/BuildCraft/BuildCraft/milestones{/number}","notifications_url":"https://api.github.com/repos/BuildCraft/BuildCraft/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/BuildCraft/BuildCraft/labels{/name}","releases_url":"https://api.github.com/repos/BuildCraft/BuildCraft/releases{/id}","created_at":"2012-04-16T10:57:39Z","updated_at":"2014-06-25T14:57:05Z","pushed_at":"2014-06-29T14:58:48Z","git_url":"git://github.com/BuildCraft/BuildCraft.git","ssh_url":"git@github.com:BuildCraft/BuildCraft.git","clone_url":"https://github.com/BuildCraft/BuildCraft.git","svn_url":"https://github.com/BuildCraft/BuildCraft","homepage":"http://www.mod-buildcraft.com","size":31662,"stargazers_count":668,"watchers_count":668,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":false,"forks_count":411,"mirror_url":null,"open_issues_count":40,"forks":411,"open_issues":40,"watchers":668,"default_branch":"6.0.x"}},"_links":{"self":{"href":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/1787"},"html":{"href":"https://github.com/BuildCraft/BuildCraft/pull/1787"},"issue":{"href":"https://api.github.com/repos/BuildCraft/BuildCraft/issues/1787"},"comments":{"href":"https://api.github.com/repos/BuildCraft/BuildCraft/issues/1787/comments"},"review_comments":{"href":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/1787/comments"},"review_comment":{"href":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/BuildCraft/BuildCraft/pulls/1787/commits"},"statuses":{"href":"https://api.github.com/repos/BuildCraft/BuildCraft/statuses/69aa89e7aff3abb02baa0764167741d00bccecac"}}}}