GitHub PullRequestReviewCommentEvent at 2014-11-24 14:15:16
Repository: rock-webapp
Actor: planthaber
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/planthaber/rock-webapp/pulls/comments/20791535","id":20791535,"diff_hunk":"@@ -40,7 +50,11 @@ def self.stream_async_data_to_websocket(env, data_source, count = Float::INFINIT\n                     end\n                     get do\n                         if params[:extended_info]\n-                            Hash[tasks: Orocos.name_service.each_task.map(&:to_h)]","path":"lib/rock/webapp/tasks/api.rb","position":36,"original_position":36,"commit_id":"41c77d39440a332030ae30efac98a68c9b692809","original_commit_id":"41c77d39440a332030ae30efac98a68c9b692809","user":{"login":"planthaber","id":7943934,"avatar_url":"https://avatars.githubusercontent.com/u/7943934?v=3","gravatar_id":"","url":"https://api.github.com/users/planthaber","html_url":"https://github.com/planthaber","followers_url":"https://api.github.com/users/planthaber/followers","following_url":"https://api.github.com/users/planthaber/following{/other_user}","gists_url":"https://api.github.com/users/planthaber/gists{/gist_id}","starred_url":"https://api.github.com/users/planthaber/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/planthaber/subscriptions","organizations_url":"https://api.github.com/users/planthaber/orgs","repos_url":"https://api.github.com/users/planthaber/repos","events_url":"https://api.github.com/users/planthaber/events{/privacy}","received_events_url":"https://api.github.com/users/planthaber/received_events","type":"User","site_admin":false},"body":"retried the original code and also my proposals here:\r\n\r\nUnexpected error while processing request: no block given (yield)\r\n\t/home/planthaber/dfki/rock/transterra/tools/orocos.rb/lib/orocos/name_service.rb:174:in `block in each_task'\r\n\t/home/planthaber/dfki/rock/transterra/tools/orocos.rb/lib/orocos/name_service.rb:169:in `each'\r\n\t/home/planthaber/dfki/rock/transterra/tools/orocos.rb/lib/orocos/name_service.rb:169:in `each_task'\r\n\t/home/planthaber/dfki/rock/transterra/tools/webapp/lib/rock/webapp/tasks/api.rb:60:in `block (2 levels) in <class:API>'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/endpoint.rb:45:in `call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/endpoint.rb:45:in `block in generate_api_method'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/endpoint.rb:220:in `call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/endpoint.rb:220:in `run'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/endpoint.rb:171:in `block in call!'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/middleware/base.rb:24:in `call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/middleware/base.rb:24:in `call!'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/middleware/base.rb:18:in `call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/middleware/base.rb:24:in `call!'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/middleware/base.rb:18:in `call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/middleware/error.rb:27:in `block in call!'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/middleware/error.rb:26:in `catch'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/middleware/error.rb:26:in `call!'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/middleware/base.rb:18:in `call'\r\n\t/usr/lib/ruby/vendor_ruby/rack/head.rb:11:in `call'\r\n\t/usr/lib/ruby/vendor_ruby/rack/builder.rb:138:in `call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/endpoint.rb:172:in `call!'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/endpoint.rb:160:in `call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/rack-mount-0.8.3/lib/rack/mount/route_set.rb:152:in `block in call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/rack-mount-0.8.3/lib/rack/mount/code_generation.rb:96:in `block in recognize'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/rack-mount-0.8.3/lib/rack/mount/code_generation.rb:68:in `optimized_each'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/rack-mount-0.8.3/lib/rack/mount/code_generation.rb:95:in `recognize'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/rack-mount-0.8.3/lib/rack/mount/route_set.rb:141:in `call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/api.rb:128:in `call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/api.rb:43:in `call!'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/grape-0.9.0/lib/grape/api.rb:39:in `call'\r\n\t/usr/lib/ruby/vendor_ruby/rack/builder.rb:138:in `call'\r\n\t/usr/lib/ruby/vendor_ruby/rack/urlmap.rb:65:in `block in call'\r\n\t/usr/lib/ruby/vendor_ruby/rack/urlmap.rb:50:in `each'\r\n\t/usr/lib/ruby/vendor_ruby/rack/urlmap.rb:50:in `call'\r\n\t/usr/lib/ruby/vendor_ruby/rack/builder.rb:138:in `call'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/thin-1.6.2/lib/thin/connection.rb:86:in `block in pre_process'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/thin-1.6.2/lib/thin/connection.rb:84:in `catch'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/thin-1.6.2/lib/thin/connection.rb:84:in `pre_process'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/thin-1.6.2/lib/thin/connection.rb:53:in `process'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/faye-websocket-0.7.4/lib/faye/adapters/thin.rb:40:in `process'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/thin-1.6.2/lib/thin/connection.rb:39:in `receive_data'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/faye-websocket-0.7.4/lib/faye/adapters/thin.rb:44:in `receive_data'\r\n\t/usr/lib/ruby/vendor_ruby/eventmachine.rb:187:in `run_machine'\r\n\t/usr/lib/ruby/vendor_ruby/eventmachine.rb:187:in `run'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/thin-1.6.2/lib/thin/backends/base.rb:73:in `start'\r\n\t/home/planthaber/dfki/rock/transterra/.gems/gems/thin-1.6.2/lib/thin/server.rb:162:in `start'\r\n\t/usr/lib/ruby/vendor_ruby/rack/handler/thin.rb:16:in `run'\r\n\t/home/planthaber/dfki/rock/transterra/tools/webapp/bin/rock-webapp:101:in `<main>'\r\n","created_at":"2014-11-24T14:15:16Z","updated_at":"2014-11-24T14:15:16Z","html_url":"https://github.com/planthaber/rock-webapp/pull/17#discussion_r20791535","pull_request_url":"https://api.github.com/repos/planthaber/rock-webapp/pulls/17","_links":{"self":{"href":"https://api.github.com/repos/planthaber/rock-webapp/pulls/comments/20791535"},"html":{"href":"https://github.com/planthaber/rock-webapp/pull/17#discussion_r20791535"},"pull_request":{"href":"https://api.github.com/repos/planthaber/rock-webapp/pulls/17"}}},"pull_request":{"url":"https://api.github.com/repos/planthaber/rock-webapp/pulls/17","id":21934290,"html_url":"https://github.com/planthaber/rock-webapp/pull/17","diff_url":"https://github.com/planthaber/rock-webapp/pull/17.diff","patch_url":"https://github.com/planthaber/rock-webapp/pull/17.patch","issue_url":"https://api.github.com/repos/planthaber/rock-webapp/issues/17","number":17,"state":"closed","locked":false,"title":"Buffered Ports ","user":{"login":"planthaber","id":7943934,"avatar_url":"https://avatars.githubusercontent.com/u/7943934?v=3","gravatar_id":"","url":"https://api.github.com/users/planthaber","html_url":"https://github.com/planthaber","followers_url":"https://api.github.com/users/planthaber/followers","following_url":"https://api.github.com/users/planthaber/following{/other_user}","gists_url":"https://api.github.com/users/planthaber/gists{/gist_id}","starred_url":"https://api.github.com/users/planthaber/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/planthaber/subscriptions","organizations_url":"https://api.github.com/users/planthaber/orgs","repos_url":"https://api.github.com/users/planthaber/repos","events_url":"https://api.github.com/users/planthaber/events{/privacy}","received_events_url":"https://api.github.com/users/planthaber/received_events","type":"User","site_admin":false},"body":"Re-issue of pull request https://github.com/planthaber/rock-webapp/pull/11 against master","created_at":"2014-09-29T13:32:57Z","updated_at":"2014-11-24T14:15:16Z","closed_at":"2014-11-20T14:27:48Z","merged_at":"2014-11-20T14:27:48Z","merge_commit_sha":"89b7a86f32eb4cc291607735b684d45988439238","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/planthaber/rock-webapp/pulls/17/commits","review_comments_url":"https://api.github.com/repos/planthaber/rock-webapp/pulls/17/comments","review_comment_url":"https://api.github.com/repos/planthaber/rock-webapp/pulls/comments/{number}","comments_url":"https://api.github.com/repos/planthaber/rock-webapp/issues/17/comments","statuses_url":"https://api.github.com/repos/planthaber/rock-webapp/statuses/41c77d39440a332030ae30efac98a68c9b692809","head":{"label":"planthaber:syskit_writer","ref":"syskit_writer","sha":"41c77d39440a332030ae30efac98a68c9b692809","user":{"login":"planthaber","id":7943934,"avatar_url":"https://avatars.githubusercontent.com/u/7943934?v=3","gravatar_id":"","url":"https://api.github.com/users/planthaber","html_url":"https://github.com/planthaber","followers_url":"https://api.github.com/users/planthaber/followers","following_url":"https://api.github.com/users/planthaber/following{/other_user}","gists_url":"https://api.github.com/users/planthaber/gists{/gist_id}","starred_url":"https://api.github.com/users/planthaber/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/planthaber/subscriptions","organizations_url":"https://api.github.com/users/planthaber/orgs","repos_url":"https://api.github.com/users/planthaber/repos","events_url":"https://api.github.com/users/planthaber/events{/privacy}","received_events_url":"https://api.github.com/users/planthaber/received_events","type":"User","site_admin":false},"repo":{"id":22064337,"name":"rock-webapp","full_name":"planthaber/rock-webapp","owner":{"login":"planthaber","id":7943934,"avatar_url":"https://avatars.githubusercontent.com/u/7943934?v=3","gravatar_id":"","url":"https://api.github.com/users/planthaber","html_url":"https://github.com/planthaber","followers_url":"https://api.github.com/users/planthaber/followers","following_url":"https://api.github.com/users/planthaber/following{/other_user}","gists_url":"https://api.github.com/users/planthaber/gists{/gist_id}","starred_url":"https://api.github.com/users/planthaber/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/planthaber/subscriptions","organizations_url":"https://api.github.com/users/planthaber/orgs","repos_url":"https://api.github.com/users/planthaber/repos","events_url":"https://api.github.com/users/planthaber/events{/privacy}","received_events_url":"https://api.github.com/users/planthaber/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/planthaber/rock-webapp","description":"HTTP API for the Robot Construction Kit (Rock)","fork":false,"url":"https://api.github.com/repos/planthaber/rock-webapp","forks_url":"https://api.github.com/repos/planthaber/rock-webapp/forks","keys_url":"https://api.github.com/repos/planthaber/rock-webapp/keys{/key_id}","collaborators_url":"https://api.github.com/repos/planthaber/rock-webapp/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/planthaber/rock-webapp/teams","hooks_url":"https://api.github.com/repos/planthaber/rock-webapp/hooks","issue_events_url":"https://api.github.com/repos/planthaber/rock-webapp/issues/events{/number}","events_url":"https://api.github.com/repos/planthaber/rock-webapp/events","assignees_url":"https://api.github.com/repos/planthaber/rock-webapp/assignees{/user}","branches_url":"https://api.github.com/repos/planthaber/rock-webapp/branches{/branch}","tags_url":"https://api.github.com/repos/planthaber/rock-webapp/tags","blobs_url":"https://api.github.com/repos/planthaber/rock-webapp/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/planthaber/rock-webapp/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/planthaber/rock-webapp/git/refs{/sha}","trees_url":"https://api.github.com/repos/planthaber/rock-webapp/git/trees{/sha}","statuses_url":"https://api.github.com/repos/planthaber/rock-webapp/statuses/{sha}","languages_url":"https://api.github.com/repos/planthaber/rock-webapp/languages","stargazers_url":"https://api.github.com/repos/planthaber/rock-webapp/stargazers","contributors_url":"https://api.github.com/repos/planthaber/rock-webapp/contributors","subscribers_url":"https://api.github.com/repos/planthaber/rock-webapp/subscribers","subscription_url":"https://api.github.com/repos/planthaber/rock-webapp/subscription","commits_url":"https://api.github.com/repos/planthaber/rock-webapp/commits{/sha}","git_commits_url":"https://api.github.com/repos/planthaber/rock-webapp/git/commits{/sha}","comments_url":"https://api.github.com/repos/planthaber/rock-webapp/comments{/number}","issue_comment_url":"https://api.github.com/repos/planthaber/rock-webapp/issues/comments/{number}","contents_url":"https://api.github.com/repos/planthaber/rock-webapp/contents/{+path}","compare_url":"https://api.github.com/repos/planthaber/rock-webapp/compare/{base}...{head}","merges_url":"https://api.github.com/repos/planthaber/rock-webapp/merges","archive_url":"https://api.github.com/repos/planthaber/rock-webapp/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/planthaber/rock-webapp/downloads","issues_url":"https://api.github.com/repos/planthaber/rock-webapp/issues{/number}","pulls_url":"https://api.github.com/repos/planthaber/rock-webapp/pulls{/number}","milestones_url":"https://api.github.com/repos/planthaber/rock-webapp/milestones{/number}","notifications_url":"https://api.github.com/repos/planthaber/rock-webapp/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/planthaber/rock-webapp/labels{/name}","releases_url":"https://api.github.com/repos/planthaber/rock-webapp/releases{/id}","created_at":"2014-07-21T13:01:08Z","updated_at":"2014-11-20T14:37:47Z","pushed_at":"2014-11-20T14:37:47Z","git_url":"git://github.com/planthaber/rock-webapp.git","ssh_url":"git@github.com:planthaber/rock-webapp.git","clone_url":"https://github.com/planthaber/rock-webapp.git","svn_url":"https://github.com/planthaber/rock-webapp","homepage":"","size":1765,"stargazers_count":0,"watchers_count":0,"language":"CSS","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":2,"mirror_url":null,"open_issues_count":0,"forks":2,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"planthaber:master","ref":"master","sha":"92087e27137ff948341e767534c3472a720828cf","user":{"login":"planthaber","id":7943934,"avatar_url":"https://avatars.githubusercontent.com/u/7943934?v=3","gravatar_id":"","url":"https://api.github.com/users/planthaber","html_url":"https://github.com/planthaber","followers_url":"https://api.github.com/users/planthaber/followers","following_url":"https://api.github.com/users/planthaber/following{/other_user}","gists_url":"https://api.github.com/users/planthaber/gists{/gist_id}","starred_url":"https://api.github.com/users/planthaber/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/planthaber/subscriptions","organizations_url":"https://api.github.com/users/planthaber/orgs","repos_url":"https://api.github.com/users/planthaber/repos","events_url":"https://api.github.com/users/planthaber/events{/privacy}","received_events_url":"https://api.github.com/users/planthaber/received_events","type":"User","site_admin":false},"repo":{"id":22064337,"name":"rock-webapp","full_name":"planthaber/rock-webapp","owner":{"login":"planthaber","id":7943934,"avatar_url":"https://avatars.githubusercontent.com/u/7943934?v=3","gravatar_id":"","url":"https://api.github.com/users/planthaber","html_url":"https://github.com/planthaber","followers_url":"https://api.github.com/users/planthaber/followers","following_url":"https://api.github.com/users/planthaber/following{/other_user}","gists_url":"https://api.github.com/users/planthaber/gists{/gist_id}","starred_url":"https://api.github.com/users/planthaber/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/planthaber/subscriptions","organizations_url":"https://api.github.com/users/planthaber/orgs","repos_url":"https://api.github.com/users/planthaber/repos","events_url":"https://api.github.com/users/planthaber/events{/privacy}","received_events_url":"https://api.github.com/users/planthaber/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/planthaber/rock-webapp","description":"HTTP API for the Robot Construction Kit (Rock)","fork":false,"url":"https://api.github.com/repos/planthaber/rock-webapp","forks_url":"https://api.github.com/repos/planthaber/rock-webapp/forks","keys_url":"https://api.github.com/repos/planthaber/rock-webapp/keys{/key_id}","collaborators_url":"https://api.github.com/repos/planthaber/rock-webapp/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/planthaber/rock-webapp/teams","hooks_url":"https://api.github.com/repos/planthaber/rock-webapp/hooks","issue_events_url":"https://api.github.com/repos/planthaber/rock-webapp/issues/events{/number}","events_url":"https://api.github.com/repos/planthaber/rock-webapp/events","assignees_url":"https://api.github.com/repos/planthaber/rock-webapp/assignees{/user}","branches_url":"https://api.github.com/repos/planthaber/rock-webapp/branches{/branch}","tags_url":"https://api.github.com/repos/planthaber/rock-webapp/tags","blobs_url":"https://api.github.com/repos/planthaber/rock-webapp/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/planthaber/rock-webapp/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/planthaber/rock-webapp/git/refs{/sha}","trees_url":"https://api.github.com/repos/planthaber/rock-webapp/git/trees{/sha}","statuses_url":"https://api.github.com/repos/planthaber/rock-webapp/statuses/{sha}","languages_url":"https://api.github.com/repos/planthaber/rock-webapp/languages","stargazers_url":"https://api.github.com/repos/planthaber/rock-webapp/stargazers","contributors_url":"https://api.github.com/repos/planthaber/rock-webapp/contributors","subscribers_url":"https://api.github.com/repos/planthaber/rock-webapp/subscribers","subscription_url":"https://api.github.com/repos/planthaber/rock-webapp/subscription","commits_url":"https://api.github.com/repos/planthaber/rock-webapp/commits{/sha}","git_commits_url":"https://api.github.com/repos/planthaber/rock-webapp/git/commits{/sha}","comments_url":"https://api.github.com/repos/planthaber/rock-webapp/comments{/number}","issue_comment_url":"https://api.github.com/repos/planthaber/rock-webapp/issues/comments/{number}","contents_url":"https://api.github.com/repos/planthaber/rock-webapp/contents/{+path}","compare_url":"https://api.github.com/repos/planthaber/rock-webapp/compare/{base}...{head}","merges_url":"https://api.github.com/repos/planthaber/rock-webapp/merges","archive_url":"https://api.github.com/repos/planthaber/rock-webapp/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/planthaber/rock-webapp/downloads","issues_url":"https://api.github.com/repos/planthaber/rock-webapp/issues{/number}","pulls_url":"https://api.github.com/repos/planthaber/rock-webapp/pulls{/number}","milestones_url":"https://api.github.com/repos/planthaber/rock-webapp/milestones{/number}","notifications_url":"https://api.github.com/repos/planthaber/rock-webapp/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/planthaber/rock-webapp/labels{/name}","releases_url":"https://api.github.com/repos/planthaber/rock-webapp/releases{/id}","created_at":"2014-07-21T13:01:08Z","updated_at":"2014-11-20T14:37:47Z","pushed_at":"2014-11-20T14:37:47Z","git_url":"git://github.com/planthaber/rock-webapp.git","ssh_url":"git@github.com:planthaber/rock-webapp.git","clone_url":"https://github.com/planthaber/rock-webapp.git","svn_url":"https://github.com/planthaber/rock-webapp","homepage":"","size":1765,"stargazers_count":0,"watchers_count":0,"language":"CSS","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":2,"mirror_url":null,"open_issues_count":0,"forks":2,"open_issues":0,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/planthaber/rock-webapp/pulls/17"},"html":{"href":"https://github.com/planthaber/rock-webapp/pull/17"},"issue":{"href":"https://api.github.com/repos/planthaber/rock-webapp/issues/17"},"comments":{"href":"https://api.github.com/repos/planthaber/rock-webapp/issues/17/comments"},"review_comments":{"href":"https://api.github.com/repos/planthaber/rock-webapp/pulls/17/comments"},"review_comment":{"href":"https://api.github.com/repos/planthaber/rock-webapp/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/planthaber/rock-webapp/pulls/17/commits"},"statuses":{"href":"https://api.github.com/repos/planthaber/rock-webapp/statuses/41c77d39440a332030ae30efac98a68c9b692809"}}}}