GitHub PullRequestReviewCommentEvent at 2014-01-21 15:51:11
Repository: zentyal
Actor: jacalvo
Payload: {"comment":{"url":"https://api.github.com/repos/Zentyal/zentyal/pulls/comments/9041136","id":9041136,"diff_hunk":"@@ -26,6 +26,22 @@ use EBox::Gettext;\n use EBox::Types::DomainName;\n use EBox::Types::Boolean;\n use EBox::Validate;\n+use Socket;\n+\n+# Method: validateRow\n+#\n+# Overrides:\n+#\n+#       <EBox::Model::DataTable::validateRow>\n+#\n+sub validateRow","path":"main/squid/src/EBox/Squid/Model/TransparentExceptions.pm","position":18,"original_position":18,"commit_id":"fc340b597f758081ab833d851282157a998fbf1c","original_commit_id":"fc340b597f758081ab833d851282157a998fbf1c","user":{"login":"jacalvo","id":491695,"avatar_url":"https://gravatar.com/avatar/4d39b8ebaeb92d9d428d1d724299220a?d=https%3A%2F%2Fidenticons.github.com%2Ff97b20080497f3d1dca9a58568a32eb8.png&r=x","gravatar_id":"4d39b8ebaeb92d9d428d1d724299220a","url":"https://api.github.com/users/jacalvo","html_url":"https://github.com/jacalvo","followers_url":"https://api.github.com/users/jacalvo/followers","following_url":"https://api.github.com/users/jacalvo/following{/other_user}","gists_url":"https://api.github.com/users/jacalvo/gists{/gist_id}","starred_url":"https://api.github.com/users/jacalvo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jacalvo/subscriptions","organizations_url":"https://api.github.com/users/jacalvo/orgs","repos_url":"https://api.github.com/users/jacalvo/repos","events_url":"https://api.github.com/users/jacalvo/events{/privacy}","received_events_url":"https://api.github.com/users/jacalvo/received_events","type":"User","site_admin":false},"body":"But what's the advantage of having the types in this case?","created_at":"2014-01-21T15:51:11Z","updated_at":"2014-01-21T15:51:11Z","html_url":"https://github.com/Zentyal/zentyal/pull/889#discussion_r9041136","pull_request_url":"https://api.github.com/repos/Zentyal/zentyal/pulls/889","_links":{"self":{"href":"https://api.github.com/repos/Zentyal/zentyal/pulls/comments/9041136"},"html":{"href":"https://github.com/Zentyal/zentyal/pull/889#discussion_r9041136"},"pull_request":{"href":"https://api.github.com/repos/Zentyal/zentyal/pulls/889"}}}}