)]}'
{"tox.ini":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"3ab59ee5c93c5da9f07c8f1183767115219074b6","unresolved":false,"context_lines":[{"line_number":88,"context_line":"# TODO(marun) H404 multi line docstring should start with a summary"},{"line_number":89,"context_line":"# TODO(dougwig) -- uncomment this to test for remaining linkages"},{"line_number":90,"context_line":"# N530 direct neutron imports not allowed"},{"line_number":91,"context_line":"ignore \u003d E125,E126,E128,E129,E265,H404,H405,N530, H904"},{"line_number":92,"context_line":"show-source \u003d true"},{"line_number":93,"context_line":"exclude \u003d .venv,.git,.tox,dist,doc,*lib/python*,.tmp,*egg,build,tools,.ropeproject,rally-scenarios"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-properties","patch_set":2,"id":"ba2be162_4780c8c0","line":91,"updated":"2017-03-02 17:51:47.000000000","message":"so we disable a check? why? if there is a valid reason, it should be described in line 90+ in a comment.","commit_id":"0f435240b6094b198950d32683b689d57d59a2ab"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"7b8a549d8f0eb7d8dfcf4da75fe62fed46062aad","unresolved":false,"context_lines":[{"line_number":88,"context_line":"# TODO(marun) H404 multi line docstring should start with a summary"},{"line_number":89,"context_line":"# TODO(dougwig) -- uncomment this to test for remaining linkages"},{"line_number":90,"context_line":"# N530 direct neutron imports not allowed"},{"line_number":91,"context_line":"ignore \u003d E125,E126,E128,E129,E265,H404,H405,N530, H904"},{"line_number":92,"context_line":"show-source \u003d true"},{"line_number":93,"context_line":"exclude \u003d .venv,.git,.tox,dist,doc,*lib/python*,.tmp,*egg,build,tools,.ropeproject,rally-scenarios"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-properties","patch_set":2,"id":"ba2be162_b2154041","line":91,"in_reply_to":"ba2be162_4780c8c0","updated":"2017-03-02 17:56:36.000000000","message":"Why is \"[H904] Delay string interpolations at logging calls \"required here?","commit_id":"0f435240b6094b198950d32683b689d57d59a2ab"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"9384825c53132936cf5201042c10b4ccb4af95bb","unresolved":false,"context_lines":[{"line_number":88,"context_line":"# TODO(marun) H404 multi line docstring should start with a summary"},{"line_number":89,"context_line":"# TODO(dougwig) -- uncomment this to test for remaining linkages"},{"line_number":90,"context_line":"# N530 direct neutron imports not allowed"},{"line_number":91,"context_line":"# H904 Delay string interpolations at logging calls"},{"line_number":92,"context_line":"ignore \u003d E125,E126,E128,E129,E265,H404,H405,N530, H904"},{"line_number":93,"context_line":"show-source \u003d true"},{"line_number":94,"context_line":"exclude \u003d .venv,.git,.tox,dist,doc,*lib/python*,.tmp,*egg,build,tools,.ropeproject,rally-scenarios"}],"source_content_type":"text/x-properties","patch_set":3,"id":"ba2be162_cd6779e7","line":91,"updated":"2017-03-02 18:16:05.000000000","message":"any plans to follow up with enabling it? a todo comment would make it clear.","commit_id":"33ab5adaf2d25490210b83bc3f91015d60412eb4"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"c5f5ee9508d4f2512d61f45669f0c241ad76b2c6","unresolved":false,"context_lines":[{"line_number":88,"context_line":"# TODO(marun) H404 multi line docstring should start with a summary"},{"line_number":89,"context_line":"# TODO(dougwig) -- uncomment this to test for remaining linkages"},{"line_number":90,"context_line":"# N530 direct neutron imports not allowed"},{"line_number":91,"context_line":"# H904 Delay string interpolations at logging calls"},{"line_number":92,"context_line":"ignore \u003d E125,E126,E128,E129,E265,H404,H405,N530, H904"},{"line_number":93,"context_line":"show-source \u003d true"},{"line_number":94,"context_line":"exclude \u003d .venv,.git,.tox,dist,doc,*lib/python*,.tmp,*egg,build,tools,.ropeproject,rally-scenarios"}],"source_content_type":"text/x-properties","patch_set":3,"id":"ba2be162_dfdabce7","line":91,"in_reply_to":"ba2be162_4dbd49b6","updated":"2017-03-02 18:54:04.000000000","message":"ok seems like https://review.openstack.org/#/c/421355/ put the enable- line but hasn\u0027t bumped hacking version, so it was noop.","commit_id":"33ab5adaf2d25490210b83bc3f91015d60412eb4"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"dbe1e0c5be9f66b24578a2375d613dd1f8c7282a","unresolved":false,"context_lines":[{"line_number":88,"context_line":"# TODO(marun) H404 multi line docstring should start with a summary"},{"line_number":89,"context_line":"# TODO(dougwig) -- uncomment this to test for remaining linkages"},{"line_number":90,"context_line":"# N530 direct neutron imports not allowed"},{"line_number":91,"context_line":"# H904 Delay string interpolations at logging calls"},{"line_number":92,"context_line":"ignore \u003d E125,E126,E128,E129,E265,H404,H405,N530, H904"},{"line_number":93,"context_line":"show-source \u003d true"},{"line_number":94,"context_line":"exclude \u003d .venv,.git,.tox,dist,doc,*lib/python*,.tmp,*egg,build,tools,.ropeproject,rally-scenarios"}],"source_content_type":"text/x-properties","patch_set":3,"id":"ba2be162_4dbd49b6","line":91,"in_reply_to":"ba2be162_cd6779e7","updated":"2017-03-02 18:25:20.000000000","message":"IIUC, this may be a helpful addition , so that we do not translate logs immediately. \nPlease correct me if I am wrong.","commit_id":"33ab5adaf2d25490210b83bc3f91015d60412eb4"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"1591f249bb5e561ec3b05c94dcf0e0b89bbeaa90","unresolved":false,"context_lines":[{"line_number":89,"context_line":"# TODO(dougwig) -- uncomment this to test for remaining linkages"},{"line_number":90,"context_line":"# N530 direct neutron imports not allowed"},{"line_number":91,"context_line":"# H904 Delay string interpolations at logging calls"},{"line_number":92,"context_line":"ignore \u003d E125,E126,E128,E129,E265,H404,H405,N530, H904"},{"line_number":93,"context_line":"show-source \u003d true"},{"line_number":94,"context_line":"exclude \u003d .venv,.git,.tox,dist,doc,*lib/python*,.tmp,*egg,build,tools,.ropeproject,rally-scenarios"},{"line_number":95,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"ba2be162_cd25b909","line":92,"range":{"start_line":92,"start_character":49,"end_line":92,"end_character":50},"updated":"2017-03-02 18:27:09.000000000","message":"what\u0027s that space char for?","commit_id":"33ab5adaf2d25490210b83bc3f91015d60412eb4"}]}