GitHub PullRequestReviewCommentEvent at 2014-04-24 18:04:54
Repository: mrburns
Actor: pmclanahan
Payload: {"comment":{"url":"https://api.github.com/repos/mozilla/mrburns/pulls/comments/11962264","id":11962264,"diff_hunk":"@@ -97,16 +112,15 @@ def get_data_for_timestamp(timestamp):\n     for continent, count in continent_totals.iteritems():\n         count = int(count)\n         issues = redis.hgetall(rkeys.SHARE_CONTINENT_ISSUES.format(continent))\n-        continent_issues[continent] = {}\n         for issue, issue_count in issues.iteritems():\n             issue_count = int(issue_count)\n             issue = data_types.types_map[issue]\n             percent = get_percent(issue_count, count)\n             continent_issues[continent][issue] = percent\n-            issue_continents[issue].append({\n-                'continent': continent,\n-                'count': percent,\n-            })\n+            for cont_dict in issue_continents[issue]:","path":"smithers/milhouse.py","position":65,"original_position":65,"commit_id":"53b5c5111c38494ec0cbcea4e7aec453c2fcedea","original_commit_id":"53b5c5111c38494ec0cbcea4e7aec453c2fcedea","user":{"login":"pmclanahan","id":19934,"avatar_url":"https://avatars.githubusercontent.com/u/19934?","gravatar_id":"f0287ea2cfd2cefcc61d774dc118a278","url":"https://api.github.com/users/pmclanahan","html_url":"https://github.com/pmclanahan","followers_url":"https://api.github.com/users/pmclanahan/followers","following_url":"https://api.github.com/users/pmclanahan/following{/other_user}","gists_url":"https://api.github.com/users/pmclanahan/gists{/gist_id}","starred_url":"https://api.github.com/users/pmclanahan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pmclanahan/subscriptions","organizations_url":"https://api.github.com/users/pmclanahan/orgs","repos_url":"https://api.github.com/users/pmclanahan/repos","events_url":"https://api.github.com/users/pmclanahan/events{/privacy}","received_events_url":"https://api.github.com/users/pmclanahan/received_events","type":"User","site_admin":false},"body":"Actually I think this is too many added loops. Let me try something else.","created_at":"2014-04-24T18:04:54Z","updated_at":"2014-04-24T18:04:54Z","html_url":"https://github.com/mozilla/mrburns/pull/121#discussion_r11962264","pull_request_url":"https://api.github.com/repos/mozilla/mrburns/pulls/121","_links":{"self":{"href":"https://api.github.com/repos/mozilla/mrburns/pulls/comments/11962264"},"html":{"href":"https://github.com/mozilla/mrburns/pull/121#discussion_r11962264"},"pull_request":{"href":"https://api.github.com/repos/mozilla/mrburns/pulls/121"}}}}