)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"be30ae6103127e18f9ae0e6f6bd1f90cd1a3c917","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"enable H237 check"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Remove the remove use of \u0027new\u0027 module by properly patching the instnace"},{"line_number":10,"context_line":"object.  Additionally, enable the H237 (module is removed in Python 3) check"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I7407600478de218a4294c4bfd774cd4cf7e1741e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"1ae5cdf2_eb56d8cd","line":9,"updated":"2014-06-16 19:12:10.000000000","message":"Nit: extra \"remove\" here.","commit_id":"e7b749ad265a0a3498d7f612cf3e5c34fc6075e4"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"36cec7eb8fa1f9f6a2f4785238e957ea129656ee","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"enable H237 check"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Remove the remove use of \u0027new\u0027 module by properly patching the instnace"},{"line_number":10,"context_line":"object.  Additionally, enable the H237 (module is removed in Python 3) check"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I7407600478de218a4294c4bfd774cd4cf7e1741e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"5a890539_f9e85518","line":9,"updated":"2014-11-19 22:18:41.000000000","message":"Nit: instance","commit_id":"1f4153889018df02c99ab5ddbfe6c75a8a4e2b16"}],"neutron/tests/unit/vmware/apiclient/test_api_eventlet_request.py":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"2f355eaa171787e6ef1eb3f4ac91baa8a1cbfc2b","unresolved":false,"context_lines":[{"line_number":112,"context_line":"            eventlet.greenthread.sleep(REQUEST_TIMEOUT * 2)"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        self.req._request_timeout \u003d REQUEST_TIMEOUT"},{"line_number":115,"context_line":"        self.req._handle_request \u003d new.instancemethod("},{"line_number":116,"context_line":"            my_handle_request, self.req, request.EventletApiRequest)"},{"line_number":117,"context_line":"        self.req.start()"},{"line_number":118,"context_line":"        self.assertIsNone(self.req.join())"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_57d35403","side":"PARENT","line":115,"updated":"2014-06-12 19:16:37.000000000","message":"I think this method tests that the timeout is raised and handled correctly, so the test certainly does something. If we cannot use new, mock could be safely used instead.","commit_id":"5c9fe2522f3fb5076e6c6e139ae327422be54294"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"1b124109f5cd6b7bb9c4cd694e46fc64af5f0b2c","unresolved":false,"context_lines":[{"line_number":112,"context_line":"            eventlet.greenthread.sleep(REQUEST_TIMEOUT * 2)"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        self.req._request_timeout \u003d REQUEST_TIMEOUT"},{"line_number":115,"context_line":"        self.req._handle_request \u003d new.instancemethod("},{"line_number":116,"context_line":"            my_handle_request, self.req, request.EventletApiRequest)"},{"line_number":117,"context_line":"        self.req.start()"},{"line_number":118,"context_line":"        self.assertIsNone(self.req.join())"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_18972f49","side":"PARENT","line":115,"in_reply_to":"1ae5cdf2_57d35403","updated":"2014-06-13 03:43:01.000000000","message":"Correct I fixed the manual mock with a proper one.","commit_id":"5c9fe2522f3fb5076e6c6e139ae327422be54294"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"f80e1680ab51839cfae5b180e865d4a1239c17ae","unresolved":false,"context_lines":[{"line_number":107,"context_line":"    def test_run_and_timeout(self):"},{"line_number":108,"context_line":"        def my_handle_request():"},{"line_number":109,"context_line":"            LOG.info(\u0027my_handle_request() self: %s\u0027 % self.req)"},{"line_number":110,"context_line":"            LOG.info(\u0027my_handle_request() dir(self): %s\u0027 % dir(self.req))"},{"line_number":111,"context_line":"            eventlet.greenthread.sleep(REQUEST_TIMEOUT * 2)"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        with mock.patch.object("}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_38ceab5a","line":110,"range":{"start_line":110,"start_character":46,"end_line":110,"end_character":50},"updated":"2014-06-13 03:50:44.000000000","message":"\u0027self\u0027 in messages are now misleading.  how about \u0027req\u0027 ?","commit_id":"07b659b4717d70258c83b41591d6d8af9758e8de"}],"tox.ini":[{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"c55d4fb917e18106fd3183957da55f24b42c6655","unresolved":false,"context_lines":[{"line_number":65,"context_line":"# E128 continuation line under-indented for visual indent"},{"line_number":66,"context_line":"# E129 visually indented line with same indent as next logical line"},{"line_number":67,"context_line":"# E265 block comment should start with ‘# ‘"},{"line_number":68,"context_line":"# H237 module is removed in Python 3"},{"line_number":69,"context_line":"# H305 imports not grouped correctly"},{"line_number":70,"context_line":"# H307 like imports should be grouped together"},{"line_number":71,"context_line":"# H401 docstring should not start with a space"}],"source_content_type":"text/x-properties","patch_set":8,"id":"5a890539_2d11aeac","line":68,"updated":"2014-11-20 05:08:13.000000000","message":"Remove this line too.","commit_id":"1f4153889018df02c99ab5ddbfe6c75a8a4e2b16"}]}