GitHub PullRequestReviewCommentEvent at 2014-03-12 19:35:56
Repository: strands_navigation
Actor: BFALacerda
Payload: {"comment":{"url":"https://api.github.com/repos/strands-project/strands_navigation/pulls/comments/10536300","id":10536300,"diff_hunk":"@@ -85,6 +70,8 @@ def __init__(self, name, filename, mode) :\n         config = self.client.get_configuration()\n         self.dyt = config['yaw_goal_tolerance']\n \n+        self.stats_pub = rospy.Publisher('/TopologicalNavigation/Statistics', NavStatistics)","path":"topological_navigation/scripts/navigation.py","position":44,"original_position":44,"commit_id":"d8a07c9ec78e0eac1b7ce31581190e0a7cfd9f2a","original_commit_id":"d8a07c9ec78e0eac1b7ce31581190e0a7cfd9f2a","user":{"login":"BFALacerda","id":4320588,"avatar_url":"https://gravatar.com/avatar/6af22ce0f61c6ac7bbf2832097e823e9?d=https%3A%2F%2Fidenticons.github.com%2F3b0347d073dc465f8d1a76666db607a7.png&r=x","gravatar_id":"6af22ce0f61c6ac7bbf2832097e823e9","url":"https://api.github.com/users/BFALacerda","html_url":"https://github.com/BFALacerda","followers_url":"https://api.github.com/users/BFALacerda/followers","following_url":"https://api.github.com/users/BFALacerda/following{/other_user}","gists_url":"https://api.github.com/users/BFALacerda/gists{/gist_id}","starred_url":"https://api.github.com/users/BFALacerda/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BFALacerda/subscriptions","organizations_url":"https://api.github.com/users/BFALacerda/orgs","repos_url":"https://api.github.com/users/BFALacerda/repos","events_url":"https://api.github.com/users/BFALacerda/events{/privacy}","received_events_url":"https://api.github.com/users/BFALacerda/received_events","type":"User","site_admin":false},"body":"just nitpicking here: this is not following the naming conventions for topics, it should be /topological_navigation/statistics","created_at":"2014-03-12T19:35:56Z","updated_at":"2014-03-12T19:35:56Z","html_url":"https://github.com/strands-project/strands_navigation/pull/26#discussion_r10536300","pull_request_url":"https://api.github.com/repos/strands-project/strands_navigation/pulls/26","_links":{"self":{"href":"https://api.github.com/repos/strands-project/strands_navigation/pulls/comments/10536300"},"html":{"href":"https://github.com/strands-project/strands_navigation/pull/26#discussion_r10536300"},"pull_request":{"href":"https://api.github.com/repos/strands-project/strands_navigation/pulls/26"}}}}