{"action":"created","comment":{"url":"https://api.github.com/repos/translate/pootle/pulls/comments/46120028","id":46120028,"diff_hunk":"@@ -503,11 +504,11 @@ def get_units(request):\n                 break\n \n         if sort_by_field is None or sort_on == 'units':\n-            # Since `extra()` has been used before, it's necessary to explicitly\n-            # request the `store__pootle_path` field. This is a subtetly in\n-            # Django's ORM.\n-            uid_list = [u['id'] for u in step_queryset.values('id',\n-                                                              'store__pootle_path')]\n+            # Since `extra()` has been used before, it's necessary to\n+            # explicitly request the `store__pootle_path` field. This is a\n+            # subtetly in Django's ORM.","path":"pootle/apps/pootle_store/views.py","position":99,"original_position":99,"commit_id":"356afea54116e83e1faca65f24cf1f3fe758db02","original_commit_id":"356afea54116e83e1faca65f24cf1f3fe758db02","user":{"login":"unho","id":939880,"avatar_url":"https://avatars.githubusercontent.com/u/939880?v=3","gravatar_id":"","url":"https://api.github.com/users/unho","html_url":"https://github.com/unho","followers_url":"https://api.github.com/users/unho/followers","following_url":"https://api.github.com/users/unho/following{/other_user}","gists_url":"https://api.github.com/users/unho/gists{/gist_id}","starred_url":"https://api.github.com/users/unho/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/unho/subscriptions","organizations_url":"https://api.github.com/users/unho/orgs","repos_url":"https://api.github.com/users/unho/repos","events_url":"https://api.github.com/users/unho/events{/privacy}","received_events_url":"https://api.github.com/users/unho/received_events","type":"User","site_admin":false},"body":"Typo in \"subtetly\"?","created_at":"2015-11-30T09:08:09Z","updated_at":"2015-11-30T09:08:09Z","html_url":"https://github.com/translate/pootle/pull/4245#discussion_r46120028","pull_request_url":"https://api.github.com/repos/translate/pootle/pulls/4245","_links":{"self":{"href":"https://api.github.com/repos/translate/pootle/pulls/comments/46120028"},"html":{"href":"https://github.com/translate/pootle/pull/4245#discussion_r46120028"},"pull_request":{"href":"https://api.github.com/repos/translate/pootle/pulls/4245"}}},"pull_request":{"url":"https://api.github.com/repos/translate/pootle/pulls/4245","id":52047503,"html_url":"https://github.com/translate/pootle/pull/4245","diff_url":"https://github.com/translate/pootle/pull/4245.diff","patch_url":"https://github.com/translate/pootle/pull/4245.patch","issue_url":"https://api.github.com/repos/translate/pootle/issues/4245","number":4245,"state":"open","locked":false,"title":"Improving PEP8 performance","user":{"login":"dwaynebailey","id":647438,"avatar_url":"https://avatars.githubusercontent.com/u/647438?v=3","gravatar_id":"","url":"https://api.github.com/users/dwaynebailey","html_url":"https://github.com/dwaynebailey","followers_url":"https://api.github.com/users/dwaynebailey/followers","following_url":"https://api.github.com/users/dwaynebailey/following{/other_user}","gists_url":"https://api.github.com/users/dwaynebailey/gists{/gist_id}","starred_url":"https://api.github.com/users/dwaynebailey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dwaynebailey/subscriptions","organizations_url":"https://api.github.com/users/dwaynebailey/orgs","repos_url":"https://api.github.com/users/dwaynebailey/repos","events_url":"https://api.github.com/users/dwaynebailey/events{/privacy}","received_events_url":"https://api.github.com/users/dwaynebailey/received_events","type":"User","site_admin":false},"body":"This increases our PEP8 coverage as follows:\r\n\r\n- One issue is taken down to zero (ensuring we don't regress but not worrying too much about creating new errors that we don't block on)\r\n- Add checks that we've cleaned into our travis pep8 check\r\n\r\nWe got to this coverage:\r\n```\r\n92      E265 block comment should start with '# '\r\n6       E302 expected 2 blank lines, found 0\r\n45      E501 line too long (91 > 84 characters)\r\n```","created_at":"2015-11-28T20:11:25Z","updated_at":"2015-11-30T09:08:09Z","closed_at":null,"merged_at":null,"merge_commit_sha":"bf1611581f6a5ec1c617b635dbc0f884dd4896ea","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/translate/pootle/pulls/4245/commits","review_comments_url":"https://api.github.com/repos/translate/pootle/pulls/4245/comments","review_comment_url":"https://api.github.com/repos/translate/pootle/pulls/comments{/number}","comments_url":"https://api.github.com/repos/translate/pootle/issues/4245/comments","statuses_url":"https://api.github.com/repos/translate/pootle/statuses/356afea54116e83e1faca65f24cf1f3fe758db02","head":{"label":"dwaynebailey:fix/pep8-to-zero","ref":"fix/pep8-to-zero","sha":"356afea54116e83e1faca65f24cf1f3fe758db02","user":{"login":"dwaynebailey","id":647438,"avatar_url":"https://avatars.githubusercontent.com/u/647438?v=3","gravatar_id":"","url":"https://api.github.com/users/dwaynebailey","html_url":"https://github.com/dwaynebailey","followers_url":"https://api.github.com/users/dwaynebailey/followers","following_url":"https://api.github.com/users/dwaynebailey/following{/other_user}","gists_url":"https://api.github.com/users/dwaynebailey/gists{/gist_id}","starred_url":"https://api.github.com/users/dwaynebailey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dwaynebailey/subscriptions","organizations_url":"https://api.github.com/users/dwaynebailey/orgs","repos_url":"https://api.github.com/users/dwaynebailey/repos","events_url":"https://api.github.com/users/dwaynebailey/events{/privacy}","received_events_url":"https://api.github.com/users/dwaynebailey/received_events","type":"User","site_admin":false},"repo":{"id":10171405,"name":"pootle","full_name":"dwaynebailey/pootle","owner":{"login":"dwaynebailey","id":647438,"avatar_url":"https://avatars.githubusercontent.com/u/647438?v=3","gravatar_id":"","url":"https://api.github.com/users/dwaynebailey","html_url":"https://github.com/dwaynebailey","followers_url":"https://api.github.com/users/dwaynebailey/followers","following_url":"https://api.github.com/users/dwaynebailey/following{/other_user}","gists_url":"https://api.github.com/users/dwaynebailey/gists{/gist_id}","starred_url":"https://api.github.com/users/dwaynebailey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dwaynebailey/subscriptions","organizations_url":"https://api.github.com/users/dwaynebailey/orgs","repos_url":"https://api.github.com/users/dwaynebailey/repos","events_url":"https://api.github.com/users/dwaynebailey/events{/privacy}","received_events_url":"https://api.github.com/users/dwaynebailey/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/dwaynebailey/pootle","description":"Online translation tool","fork":true,"url":"https://api.github.com/repos/dwaynebailey/pootle","forks_url":"https://api.github.com/repos/dwaynebailey/pootle/forks","keys_url":"https://api.github.com/repos/dwaynebailey/pootle/keys{/key_id}","collaborators_url":"https://api.github.com/repos/dwaynebailey/pootle/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/dwaynebailey/pootle/teams","hooks_url":"https://api.github.com/repos/dwaynebailey/pootle/hooks","issue_events_url":"https://api.github.com/repos/dwaynebailey/pootle/issues/events{/number}","events_url":"https://api.github.com/repos/dwaynebailey/pootle/events","assignees_url":"https://api.github.com/repos/dwaynebailey/pootle/assignees{/user}","branches_url":"https://api.github.com/repos/dwaynebailey/pootle/branches{/branch}","tags_url":"https://api.github.com/repos/dwaynebailey/pootle/tags","blobs_url":"https://api.github.com/repos/dwaynebailey/pootle/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/dwaynebailey/pootle/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/dwaynebailey/pootle/git/refs{/sha}","trees_url":"https://api.github.com/repos/dwaynebailey/pootle/git/trees{/sha}","statuses_url":"https://api.github.com/repos/dwaynebailey/pootle/statuses/{sha}","languages_url":"https://api.github.com/repos/dwaynebailey/pootle/languages","stargazers_url":"https://api.github.com/repos/dwaynebailey/pootle/stargazers","contributors_url":"https://api.github.com/repos/dwaynebailey/pootle/contributors","subscribers_url":"https://api.github.com/repos/dwaynebailey/pootle/subscribers","subscription_url":"https://api.github.com/repos/dwaynebailey/pootle/subscription","commits_url":"https://api.github.com/repos/dwaynebailey/pootle/commits{/sha}","git_commits_url":"https://api.github.com/repos/dwaynebailey/pootle/git/commits{/sha}","comments_url":"https://api.github.com/repos/dwaynebailey/pootle/comments{/number}","issue_comment_url":"https://api.github.com/repos/dwaynebailey/pootle/issues/comments{/number}","contents_url":"https://api.github.com/repos/dwaynebailey/pootle/contents/{+path}","compare_url":"https://api.github.com/repos/dwaynebailey/pootle/compare/{base}...{head}","merges_url":"https://api.github.com/repos/dwaynebailey/pootle/merges","archive_url":"https://api.github.com/repos/dwaynebailey/pootle/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/dwaynebailey/pootle/downloads","issues_url":"https://api.github.com/repos/dwaynebailey/pootle/issues{/number}","pulls_url":"https://api.github.com/repos/dwaynebailey/pootle/pulls{/number}","milestones_url":"https://api.github.com/repos/dwaynebailey/pootle/milestones{/number}","notifications_url":"https://api.github.com/repos/dwaynebailey/pootle/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/dwaynebailey/pootle/labels{/name}","releases_url":"https://api.github.com/repos/dwaynebailey/pootle/releases{/id}","created_at":"2013-05-20T11:29:39Z","updated_at":"2015-09-27T02:38:03Z","pushed_at":"2015-11-29T08:34:34Z","git_url":"git://github.com/dwaynebailey/pootle.git","ssh_url":"git@github.com:dwaynebailey/pootle.git","clone_url":"https://github.com/dwaynebailey/pootle.git","svn_url":"https://github.com/dwaynebailey/pootle","homepage":"http://pootle.translatehouse.org","size":80965,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"translate:master","ref":"master","sha":"04e7fb4b7508e4a9ea5cfa36f021266f1ebc0bed","user":{"login":"translate","id":1538178,"avatar_url":"https://avatars.githubusercontent.com/u/1538178?v=3","gravatar_id":"","url":"https://api.github.com/users/translate","html_url":"https://github.com/translate","followers_url":"https://api.github.com/users/translate/followers","following_url":"https://api.github.com/users/translate/following{/other_user}","gists_url":"https://api.github.com/users/translate/gists{/gist_id}","starred_url":"https://api.github.com/users/translate/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/translate/subscriptions","organizations_url":"https://api.github.com/users/translate/orgs","repos_url":"https://api.github.com/users/translate/repos","events_url":"https://api.github.com/users/translate/events{/privacy}","received_events_url":"https://api.github.com/users/translate/received_events","type":"Organization","site_admin":false},"repo":{"id":3769702,"name":"pootle","full_name":"translate/pootle","owner":{"login":"translate","id":1538178,"avatar_url":"https://avatars.githubusercontent.com/u/1538178?v=3","gravatar_id":"","url":"https://api.github.com/users/translate","html_url":"https://github.com/translate","followers_url":"https://api.github.com/users/translate/followers","following_url":"https://api.github.com/users/translate/following{/other_user}","gists_url":"https://api.github.com/users/translate/gists{/gist_id}","starred_url":"https://api.github.com/users/translate/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/translate/subscriptions","organizations_url":"https://api.github.com/users/translate/orgs","repos_url":"https://api.github.com/users/translate/repos","events_url":"https://api.github.com/users/translate/events{/privacy}","received_events_url":"https://api.github.com/users/translate/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/translate/pootle","description":"Online translation tool","fork":false,"url":"https://api.github.com/repos/translate/pootle","forks_url":"https://api.github.com/repos/translate/pootle/forks","keys_url":"https://api.github.com/repos/translate/pootle/keys{/key_id}","collaborators_url":"https://api.github.com/repos/translate/pootle/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/translate/pootle/teams","hooks_url":"https://api.github.com/repos/translate/pootle/hooks","issue_events_url":"https://api.github.com/repos/translate/pootle/issues/events{/number}","events_url":"https://api.github.com/repos/translate/pootle/events","assignees_url":"https://api.github.com/repos/translate/pootle/assignees{/user}","branches_url":"https://api.github.com/repos/translate/pootle/branches{/branch}","tags_url":"https://api.github.com/repos/translate/pootle/tags","blobs_url":"https://api.github.com/repos/translate/pootle/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/translate/pootle/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/translate/pootle/git/refs{/sha}","trees_url":"https://api.github.com/repos/translate/pootle/git/trees{/sha}","statuses_url":"https://api.github.com/repos/translate/pootle/statuses/{sha}","languages_url":"https://api.github.com/repos/translate/pootle/languages","stargazers_url":"https://api.github.com/repos/translate/pootle/stargazers","contributors_url":"https://api.github.com/repos/translate/pootle/contributors","subscribers_url":"https://api.github.com/repos/translate/pootle/subscribers","subscription_url":"https://api.github.com/repos/translate/pootle/subscription","commits_url":"https://api.github.com/repos/translate/pootle/commits{/sha}","git_commits_url":"https://api.github.com/repos/translate/pootle/git/commits{/sha}","comments_url":"https://api.github.com/repos/translate/pootle/comments{/number}","issue_comment_url":"https://api.github.com/repos/translate/pootle/issues/comments{/number}","contents_url":"https://api.github.com/repos/translate/pootle/contents/{+path}","compare_url":"https://api.github.com/repos/translate/pootle/compare/{base}...{head}","merges_url":"https://api.github.com/repos/translate/pootle/merges","archive_url":"https://api.github.com/repos/translate/pootle/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/translate/pootle/downloads","issues_url":"https://api.github.com/repos/translate/pootle/issues{/number}","pulls_url":"https://api.github.com/repos/translate/pootle/pulls{/number}","milestones_url":"https://api.github.com/repos/translate/pootle/milestones{/number}","notifications_url":"https://api.github.com/repos/translate/pootle/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/translate/pootle/labels{/name}","releases_url":"https://api.github.com/repos/translate/pootle/releases{/id}","created_at":"2012-03-19T23:06:03Z","updated_at":"2015-11-25T02:49:09Z","pushed_at":"2015-11-29T08:09:08Z","git_url":"git://github.com/translate/pootle.git","ssh_url":"git@github.com:translate/pootle.git","clone_url":"https://github.com/translate/pootle.git","svn_url":"https://github.com/translate/pootle","homepage":"http://pootle.translatehouse.org","size":82581,"stargazers_count":547,"watchers_count":547,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":164,"mirror_url":null,"open_issues_count":262,"forks":164,"open_issues":262,"watchers":547,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/translate/pootle/pulls/4245"},"html":{"href":"https://github.com/translate/pootle/pull/4245"},"issue":{"href":"https://api.github.com/repos/translate/pootle/issues/4245"},"comments":{"href":"https://api.github.com/repos/translate/pootle/issues/4245/comments"},"review_comments":{"href":"https://api.github.com/repos/translate/pootle/pulls/4245/comments"},"review_comment":{"href":"https://api.github.com/repos/translate/pootle/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/translate/pootle/pulls/4245/commits"},"statuses":{"href":"https://api.github.com/repos/translate/pootle/statuses/356afea54116e83e1faca65f24cf1f3fe758db02"}}}}