)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"eb154862627f279272f3b5e899f8836f1186ea98","unresolved":false,"context_lines":[{"line_number":10,"context_line":" - H301 should be enabled to avoid multiple imports for one line"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Changes:"},{"line_number":13,"context_line":" - ignoring H302(import only modules) because it appers"},{"line_number":14,"context_line":"   to checked when H301 was enabled."},{"line_number":15,"context_line":" - tox.ini updated"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_a0f01092","line":13,"updated":"2014-07-03 17:01:05.000000000","message":"appears","commit_id":"876e0004b70a3a7df245551514db0887b1b4acd5"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"0c33a912b5829f1536bbbe92b9cc6ee3a1ca1ead","unresolved":false,"context_lines":[{"line_number":10,"context_line":" - H301 should be enabled to avoid multiple imports for one line"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Changes:"},{"line_number":13,"context_line":" - ignoring H302(import only modules) because it appers"},{"line_number":14,"context_line":"   to checked when H301 was enabled."},{"line_number":15,"context_line":" - tox.ini updated"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"baada198_e32c4004","line":13,"updated":"2014-07-29 20:29:47.000000000","message":"appears.\n\nJust to be clear, H301 also enables H302, H303 and H304.","commit_id":"edbfc6d95da2189685b1a192ad5686f534b99563"}],"tox.ini":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"7c048867cc8945952d72c2aba80418fe320fb855","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"[flake8]"},{"line_number":43,"context_line":"show-source \u003d True"},{"line_number":44,"context_line":"ignore \u003d F821,H302,H306,H404"},{"line_number":45,"context_line":"builtins \u003d _"},{"line_number":46,"context_line":"exclude\u003d.venv,.tox,dist,doc,openstack,*egg,rsdns,tools,etc,build,*.po,*.pot"},{"line_number":47,"context_line":"filename\u003d*.py,trove-*"}],"source_content_type":"text/x-properties","patch_set":3,"id":"baada198_68645621","line":44,"updated":"2014-07-23 18:16:29.000000000","message":"question still remains on this; why ignore H302","commit_id":"edbfc6d95da2189685b1a192ad5686f534b99563"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"7ea79f1032d80ef203da1c03dd792e65a3af1556","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"[flake8]"},{"line_number":43,"context_line":"show-source \u003d True"},{"line_number":44,"context_line":"ignore \u003d F821,H302,H306,H404"},{"line_number":45,"context_line":"builtins \u003d _"},{"line_number":46,"context_line":"exclude\u003d.venv,.tox,dist,doc,openstack,*egg,rsdns,tools,etc,build,*.po,*.pot"},{"line_number":47,"context_line":"filename\u003d*.py,trove-*"}],"source_content_type":"text/x-properties","patch_set":3,"id":"baada198_ab2ac57b","line":44,"in_reply_to":"baada198_68645621","updated":"2014-07-24 14:23:02.000000000","message":"There\u0027s no question. H302 get enabled with H301, you can verify it manually and then re-review this patch again.\nFYI, commit message and bug-description states clearly about deps. between H301 and H302\n\nAlso see https://github.com/openstack-dev/hacking/blob/e56eddf55b4e8be9826e8eb968b2893ecab7201b/hacking/checks/imports.py","commit_id":"edbfc6d95da2189685b1a192ad5686f534b99563"}],"trove/common/auth.py":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"0c33a912b5829f1536bbbe92b9cc6ee3a1ca1ead","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import re"},{"line_number":17,"context_line":"import webob.exc"},{"line_number":18,"context_line":"from wsgi import Middleware"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from trove.common import exception"},{"line_number":21,"context_line":"from trove.openstack.common import log as logging"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_83c46461","line":18,"updated":"2014-07-29 20:29:47.000000000","message":"why this change? It isn\u0027t a H301 violation. It is H304.","commit_id":"edbfc6d95da2189685b1a192ad5686f534b99563"}]}