)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":7,"context_line":"remove usage of six library from unit tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replace six with Python 3 style code."},{"line_number":10,"context_line":"and disable M325"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I5077e71663f6b60bd774f30fcf64b36d4078cf8e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"04244925_e1b36e8b","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":16},"updated":"2021-01-27 07:40:18.000000000","message":"since this was removed entirely earlier, you can drop this","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b565b510fc39e18f76fb4ce51b4102b4a614677d","unresolved":true,"context_lines":[{"line_number":7,"context_line":"remove usage of six library from unit tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replace six with Python 3 style code."},{"line_number":10,"context_line":"and disable M325"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I5077e71663f6b60bd774f30fcf64b36d4078cf8e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"1a2f5ea2_485f6480","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":16},"updated":"2021-01-27 18:27:42.000000000","message":"remove","commit_id":"26861d28b65b62d0bdcafda061211bacb9051d0d"}],"manila/tests/api/middleware/test_faults.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"af75d893718796cc915c7a26671b7b95c91a6466","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        resp \u003d req.get_response(raiser)"},{"line_number":92,"context_line":"        self.assertEqual(\"application/json\", resp.content_type)"},{"line_number":93,"context_line":"        self.assertEqual(404, resp.status_int)"},{"line_number":94,"context_line":"        self.assertIn(\u0027whut?\u0027.encode(\"latin-1\"), resp.body)"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    def test_raise_403(self):"},{"line_number":97,"context_line":"        \"\"\"Ensure the ability to raise :class:`Fault` in WSGI-ified methods.\"\"\""}],"source_content_type":"text/x-python","patch_set":7,"id":"d4db4597_c34cd417","line":94,"range":{"start_line":94,"start_character":37,"end_line":94,"end_character":46},"updated":"2021-01-27 01:35:26.000000000","message":"utf-8","commit_id":"6b7d412412da3d42eee76ea63804a7551ac1d9f8"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"af75d893718796cc915c7a26671b7b95c91a6466","unresolved":true,"context_lines":[{"line_number":104,"context_line":"        self.assertEqual(\"application/json\", resp.content_type)"},{"line_number":105,"context_line":"        self.assertEqual(403, resp.status_int)"},{"line_number":106,"context_line":"        self.assertNotIn(\u0027resizeNotAllowed\u0027.encode(\"latin-1\"), resp.body)"},{"line_number":107,"context_line":"        self.assertIn(\u0027forbidden\u0027.encode(\"latin-1\"), resp.body)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    def test_fault_has_status_int(self):"},{"line_number":110,"context_line":"        \"\"\"Ensure the status_int is set correctly on faults.\"\"\""}],"source_content_type":"text/x-python","patch_set":7,"id":"8f96c97f_adb87421","line":107,"range":{"start_line":107,"start_character":42,"end_line":107,"end_character":49},"updated":"2021-01-27 01:35:26.000000000","message":"utf-8","commit_id":"6b7d412412da3d42eee76ea63804a7551ac1d9f8"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"eb5a2bf667e55db09dfd76757a39950f5924f3fd","unresolved":true,"context_lines":[{"line_number":104,"context_line":"        self.assertEqual(\"application/json\", resp.content_type)"},{"line_number":105,"context_line":"        self.assertEqual(403, resp.status_int)"},{"line_number":106,"context_line":"        self.assertNotIn(\u0027resizeNotAllowed\u0027.encode(\"latin-1\"), resp.body)"},{"line_number":107,"context_line":"        self.assertIn(\u0027forbidden\u0027.encode(\"latin-1\"), resp.body)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    def test_fault_has_status_int(self):"},{"line_number":110,"context_line":"        \"\"\"Ensure the status_int is set correctly on faults.\"\"\""}],"source_content_type":"text/x-python","patch_set":7,"id":"098aa45c_c470aefe","line":107,"range":{"start_line":107,"start_character":42,"end_line":107,"end_character":49},"in_reply_to":"8f96c97f_adb87421","updated":"2021-01-27 06:53:23.000000000","message":"yes, i will replace it with utf-8.","commit_id":"6b7d412412da3d42eee76ea63804a7551ac1d9f8"}],"manila/tests/api/openstack/test_wsgi.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"af75d893718796cc915c7a26671b7b95c91a6466","unresolved":true,"context_lines":[{"line_number":29,"context_line":"class RequestTest(test.TestCase):"},{"line_number":30,"context_line":"    def test_content_type_missing(self):"},{"line_number":31,"context_line":"        request \u003d wsgi.Request.blank(\u0027/tests/123\u0027, method\u003d\u0027POST\u0027)"},{"line_number":32,"context_line":"        request.body \u003d \"\u003cbody /\u003e\".encode(\"latin-1\")"},{"line_number":33,"context_line":"        self.assertIsNone(request.get_content_type())"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def test_content_type_unsupported(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"be8e7fd6_a1a144ba","line":32,"range":{"start_line":32,"start_character":42,"end_line":32,"end_character":49},"updated":"2021-01-27 01:35:26.000000000","message":"utf-8\n\n(or just drop it, utf-8 is the default)","commit_id":"6b7d412412da3d42eee76ea63804a7551ac1d9f8"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"af75d893718796cc915c7a26671b7b95c91a6466","unresolved":true,"context_lines":[{"line_number":35,"context_line":"    def test_content_type_unsupported(self):"},{"line_number":36,"context_line":"        request \u003d wsgi.Request.blank(\u0027/tests/123\u0027, method\u003d\u0027POST\u0027)"},{"line_number":37,"context_line":"        request.headers[\"Content-Type\"] \u003d \"text/html\""},{"line_number":38,"context_line":"        request.body \u003d \"asdf\u003cbr /\u003e\".encode(\"latin-1\")"},{"line_number":39,"context_line":"        self.assertRaises(exception.InvalidContentType,"},{"line_number":40,"context_line":"                          request.get_content_type)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"fa765545_b6fbdb22","line":38,"range":{"start_line":38,"start_character":44,"end_line":38,"end_character":51},"updated":"2021-01-27 01:35:26.000000000","message":"same","commit_id":"6b7d412412da3d42eee76ea63804a7551ac1d9f8"}],"manila/tests/api/v1/test_limits.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"b8cdcd07b490655d88562b7295b50ae1b7c5b5a7","unresolved":false,"context_lines":[{"line_number":18,"context_line":"\"\"\""},{"line_number":19,"context_line":"import ddt"},{"line_number":20,"context_line":"import io"},{"line_number":21,"context_line":"import http.client as http_client"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from oslo_serialization import jsonutils"},{"line_number":24,"context_line":"import webob"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_6026bf69","line":21,"updated":"2020-10-09 10:40:06.000000000","message":"pep8: H306: imports not in alphabetical order (io, http.client)","commit_id":"40c7e74eb6ed14f2ecd6cc4b6cd3b5cdbcdd12eb"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"b8cdcd07b490655d88562b7295b50ae1b7c5b5a7","unresolved":false,"context_lines":[{"line_number":381,"context_line":"                \u0027(POST, /bar*, /bar.*, 5, second);\u0027"},{"line_number":382,"context_line":"                \u0027(Say, /derp*, /derp.*, 1, day)\u0027)"},{"line_number":383,"context_line":"        except ValueError as e:"},{"line_number":384,"context_line":"            assert False, str(e)"},{"line_number":385,"context_line":""},{"line_number":386,"context_line":"        # Make sure the number of returned limits are correct"},{"line_number":387,"context_line":"        self.assertEqual(4, len(lim))"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_c01eab0b","line":384,"updated":"2020-10-09 10:40:06.000000000","message":"pep8: M325 str() and unicode() cannot be used on an exception.  Remove or use str()","commit_id":"40c7e74eb6ed14f2ecd6cc4b6cd3b5cdbcdd12eb"}],"manila/tests/api/v1/test_share_metadata.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"b8cdcd07b490655d88562b7295b50ae1b7c5b5a7","unresolved":false,"context_lines":[{"line_number":149,"context_line":"        req \u003d fakes.HTTPRequest.blank(self.url + \u0027/key1\u0027)"},{"line_number":150,"context_line":"        req.method \u003d \u0027PUT\u0027"},{"line_number":151,"context_line":"        body \u003d {\"meta\": {(\"a\" * 260): \"value1\"}}"},{"line_number":152,"context_line":"        req.body \u003d sonutils.dumps(body).encode(\"latin-1\")"},{"line_number":153,"context_line":"        req.headers[\"content-type\"] \u003d \"application/json\""},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        self.assertRaises(webob.exc.HTTPBadRequest,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_a01bb719","line":152,"updated":"2020-10-09 10:40:06.000000000","message":"pep8: F821 undefined name \u0027sonutils\u0027","commit_id":"40c7e74eb6ed14f2ecd6cc4b6cd3b5cdbcdd12eb"}],"manila/tests/api/v2/test_share_group_snapshots.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":138,"context_line":"    @ddt.unpack"},{"line_number":139,"context_line":"    def test_create(self, microversion, experimental):"},{"line_number":140,"context_line":"        fake_snap, expected_snap \u003d self._get_fake_share_group_snapshot()"},{"line_number":141,"context_line":"        fake_id \u003d str(uuidutils.generate_uuid())"},{"line_number":142,"context_line":"        body \u003d {\"share_group_snapshot\": {\"share_group_id\": fake_id}}"},{"line_number":143,"context_line":"        mock_create \u003d self.mock_object("},{"line_number":144,"context_line":"            self.controller.share_group_api, \u0027create_share_group_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"d5338aa0_8e8b1b11","line":141,"range":{"start_line":141,"start_character":18,"end_line":141,"end_character":48},"updated":"2021-01-27 07:40:18.000000000","message":"no need to type cast this, the return value from generate_uuid() is a string:\n\n \u003e\u003e\u003e type(uuidutils.generate_uuid())\n \u003cclass \u0027str\u0027\u003e","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"c088174068a576005100d4dbf516e95c6ef911dc","unresolved":false,"context_lines":[{"line_number":138,"context_line":"    @ddt.unpack"},{"line_number":139,"context_line":"    def test_create(self, microversion, experimental):"},{"line_number":140,"context_line":"        fake_snap, expected_snap \u003d self._get_fake_share_group_snapshot()"},{"line_number":141,"context_line":"        fake_id \u003d str(uuidutils.generate_uuid())"},{"line_number":142,"context_line":"        body \u003d {\"share_group_snapshot\": {\"share_group_id\": fake_id}}"},{"line_number":143,"context_line":"        mock_create \u003d self.mock_object("},{"line_number":144,"context_line":"            self.controller.share_group_api, \u0027create_share_group_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"d4f9b54e_6ece7012","line":141,"range":{"start_line":141,"start_character":18,"end_line":141,"end_character":48},"in_reply_to":"d5338aa0_8e8b1b11","updated":"2021-01-27 11:25:22.000000000","message":"Done","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":157,"context_line":"        self.assertEqual(expected_snap, res_dict[\u0027share_group_snapshot\u0027])"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"    def test_create_group_does_not_exist(self):"},{"line_number":160,"context_line":"        fake_id \u003d str(uuidutils.generate_uuid())"},{"line_number":161,"context_line":"        body \u003d {\"share_group_snapshot\": {\"share_group_id\": fake_id}}"},{"line_number":162,"context_line":"        self.mock_object("},{"line_number":163,"context_line":"            self.controller.share_group_api, \u0027create_share_group_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"494ecdf4_4ddedc43","line":160,"range":{"start_line":160,"start_character":18,"end_line":160,"end_character":21},"updated":"2021-01-27 07:40:18.000000000","message":"same here and all occurrences below","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"c088174068a576005100d4dbf516e95c6ef911dc","unresolved":false,"context_lines":[{"line_number":157,"context_line":"        self.assertEqual(expected_snap, res_dict[\u0027share_group_snapshot\u0027])"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"    def test_create_group_does_not_exist(self):"},{"line_number":160,"context_line":"        fake_id \u003d str(uuidutils.generate_uuid())"},{"line_number":161,"context_line":"        body \u003d {\"share_group_snapshot\": {\"share_group_id\": fake_id}}"},{"line_number":162,"context_line":"        self.mock_object("},{"line_number":163,"context_line":"            self.controller.share_group_api, \u0027create_share_group_snapshot\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"f311a719_21d6a06c","line":160,"range":{"start_line":160,"start_character":18,"end_line":160,"end_character":21},"in_reply_to":"494ecdf4_4ddedc43","updated":"2021-01-27 11:25:22.000000000","message":"Done","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b565b510fc39e18f76fb4ce51b4102b4a614677d","unresolved":true,"context_lines":[{"line_number":162,"context_line":"        self.mock_object("},{"line_number":163,"context_line":"            self.controller.share_group_api, \u0027create_share_group_snapshot\u0027,"},{"line_number":164,"context_line":"            mock.Mock(side_effect\u003dexception.ShareGroupNotFound("},{"line_number":165,"context_line":"                share_group_id\u003dstr(uuidutils.generate_uuid()))))"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        self.assertRaises("},{"line_number":168,"context_line":"            webob.exc.HTTPBadRequest,"}],"source_content_type":"text/x-python","patch_set":9,"id":"24bc932a_c078be9b","line":165,"range":{"start_line":165,"start_character":31,"end_line":165,"end_character":64},"updated":"2021-01-27 18:27:42.000000000","message":"please drop the type casting for this, and all other occurrences in this file. You addressed the two i pointed out in PS #8, but, there are a lot more in this file","commit_id":"26861d28b65b62d0bdcafda061211bacb9051d0d"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"430b16741aee02c4344b9ea732a1c4ce1da9208f","unresolved":false,"context_lines":[{"line_number":162,"context_line":"        self.mock_object("},{"line_number":163,"context_line":"            self.controller.share_group_api, \u0027create_share_group_snapshot\u0027,"},{"line_number":164,"context_line":"            mock.Mock(side_effect\u003dexception.ShareGroupNotFound("},{"line_number":165,"context_line":"                share_group_id\u003dstr(uuidutils.generate_uuid()))))"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        self.assertRaises("},{"line_number":168,"context_line":"            webob.exc.HTTPBadRequest,"}],"source_content_type":"text/x-python","patch_set":9,"id":"3a346e12_116d5c41","line":165,"range":{"start_line":165,"start_character":31,"end_line":165,"end_character":64},"in_reply_to":"24bc932a_c078be9b","updated":"2021-01-28 02:50:15.000000000","message":"Done","commit_id":"26861d28b65b62d0bdcafda061211bacb9051d0d"}],"manila/tests/api/v2/test_share_groups.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":50,"context_line":"        super(ShareGroupAPITest, self).setUp()"},{"line_number":51,"context_line":"        self.controller \u003d share_groups.ShareGroupController()"},{"line_number":52,"context_line":"        self.resource_name \u003d self.controller.resource_name"},{"line_number":53,"context_line":"        self.fake_share_type \u003d {\u0027id\u0027: str(uuidutils.generate_uuid())}"},{"line_number":54,"context_line":"        self.fake_share_group_type \u003d {"},{"line_number":55,"context_line":"            \u0027id\u0027: str(uuidutils.generate_uuid())}"},{"line_number":56,"context_line":"        self.api_version \u003d \u00272.34\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"2e962e1e_aac302d1","line":53,"range":{"start_line":53,"start_character":38,"end_line":53,"end_character":42},"updated":"2021-01-27 07:40:18.000000000","message":"no need to cast this, \n\nthe value from generate_uuid is a string:\n\n \u003e\u003e\u003e type(uuidutils.generate_uuid())\n \u003cclass \u0027str\u0027\u003e","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"c088174068a576005100d4dbf516e95c6ef911dc","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        super(ShareGroupAPITest, self).setUp()"},{"line_number":51,"context_line":"        self.controller \u003d share_groups.ShareGroupController()"},{"line_number":52,"context_line":"        self.resource_name \u003d self.controller.resource_name"},{"line_number":53,"context_line":"        self.fake_share_type \u003d {\u0027id\u0027: str(uuidutils.generate_uuid())}"},{"line_number":54,"context_line":"        self.fake_share_group_type \u003d {"},{"line_number":55,"context_line":"            \u0027id\u0027: str(uuidutils.generate_uuid())}"},{"line_number":56,"context_line":"        self.api_version \u003d \u00272.34\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"3d23c1d0_a506aeca","line":53,"range":{"start_line":53,"start_character":38,"end_line":53,"end_character":42},"in_reply_to":"2e962e1e_aac302d1","updated":"2021-01-27 11:25:22.000000000","message":"Done","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":52,"context_line":"        self.resource_name \u003d self.controller.resource_name"},{"line_number":53,"context_line":"        self.fake_share_type \u003d {\u0027id\u0027: str(uuidutils.generate_uuid())}"},{"line_number":54,"context_line":"        self.fake_share_group_type \u003d {"},{"line_number":55,"context_line":"            \u0027id\u0027: str(uuidutils.generate_uuid())}"},{"line_number":56,"context_line":"        self.api_version \u003d \u00272.34\u0027"},{"line_number":57,"context_line":"        self.request \u003d fakes.HTTPRequest.blank("},{"line_number":58,"context_line":"            \u0027/share-groups\u0027, version\u003dself.api_version, experimental\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":8,"id":"c5b5e1b3_060d3b6e","line":55,"range":{"start_line":55,"start_character":19,"end_line":55,"end_character":23},"updated":"2021-01-27 07:40:18.000000000","message":"same","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"c088174068a576005100d4dbf516e95c6ef911dc","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        self.resource_name \u003d self.controller.resource_name"},{"line_number":53,"context_line":"        self.fake_share_type \u003d {\u0027id\u0027: str(uuidutils.generate_uuid())}"},{"line_number":54,"context_line":"        self.fake_share_group_type \u003d {"},{"line_number":55,"context_line":"            \u0027id\u0027: str(uuidutils.generate_uuid())}"},{"line_number":56,"context_line":"        self.api_version \u003d \u00272.34\u0027"},{"line_number":57,"context_line":"        self.request \u003d fakes.HTTPRequest.blank("},{"line_number":58,"context_line":"            \u0027/share-groups\u0027, version\u003dself.api_version, experimental\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":8,"id":"d8703ba3_f327bc23","line":55,"range":{"start_line":55,"start_character":19,"end_line":55,"end_character":23},"in_reply_to":"c5b5e1b3_060d3b6e","updated":"2021-01-27 11:25:22.000000000","message":"Done","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":166,"context_line":"            req_context, self.resource_name, \u0027create\u0027)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    def test_group_create_invalid_group_snapshot_state(self):"},{"line_number":169,"context_line":"        fake_snap_id \u003d str(uuidutils.generate_uuid())"},{"line_number":170,"context_line":"        self.mock_object("},{"line_number":171,"context_line":"            self.controller.share_group_api, \u0027create\u0027,"},{"line_number":172,"context_line":"            mock.Mock(side_effect\u003dexception.InvalidShareGroupSnapshot("}],"source_content_type":"text/x-python","patch_set":8,"id":"91d8286e_2e1af0eb","line":169,"range":{"start_line":169,"start_character":23,"end_line":169,"end_character":27},"updated":"2021-01-27 07:40:18.000000000","message":"same here, and below","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"c088174068a576005100d4dbf516e95c6ef911dc","unresolved":false,"context_lines":[{"line_number":166,"context_line":"            req_context, self.resource_name, \u0027create\u0027)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"    def test_group_create_invalid_group_snapshot_state(self):"},{"line_number":169,"context_line":"        fake_snap_id \u003d str(uuidutils.generate_uuid())"},{"line_number":170,"context_line":"        self.mock_object("},{"line_number":171,"context_line":"            self.controller.share_group_api, \u0027create\u0027,"},{"line_number":172,"context_line":"            mock.Mock(side_effect\u003dexception.InvalidShareGroupSnapshot("}],"source_content_type":"text/x-python","patch_set":8,"id":"449f52cf_dcd2c932","line":169,"range":{"start_line":169,"start_character":23,"end_line":169,"end_character":27},"in_reply_to":"91d8286e_2e1af0eb","updated":"2021-01-27 11:25:22.000000000","message":"Done","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b565b510fc39e18f76fb4ce51b4102b4a614677d","unresolved":true,"context_lines":[{"line_number":398,"context_line":"            self.context, self.resource_name, \u0027create\u0027)"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"    def test_sg_create_with_source_sg_snapshot_id_and_share_network(self):"},{"line_number":401,"context_line":"        fake_snap_id \u003d str(uuidutils.generate_uuid())"},{"line_number":402,"context_line":"        fake_net_id \u003d str(uuidutils.generate_uuid())"},{"line_number":403,"context_line":"        self.mock_object(share_types, \u0027get_default_share_type\u0027,"},{"line_number":404,"context_line":"                         mock.Mock(return_value\u003dself.fake_share_type))"}],"source_content_type":"text/x-python","patch_set":9,"id":"0078ae58_ff63d83b","line":401,"range":{"start_line":401,"start_character":23,"end_line":401,"end_character":53},"updated":"2021-01-27 18:27:42.000000000","message":"please drop the type casting for this, and all other occurrences in this file. You addressed the two i pointed out in PS #8, but, there are a lot more in this file","commit_id":"26861d28b65b62d0bdcafda061211bacb9051d0d"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"430b16741aee02c4344b9ea732a1c4ce1da9208f","unresolved":false,"context_lines":[{"line_number":398,"context_line":"            self.context, self.resource_name, \u0027create\u0027)"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"    def test_sg_create_with_source_sg_snapshot_id_and_share_network(self):"},{"line_number":401,"context_line":"        fake_snap_id \u003d str(uuidutils.generate_uuid())"},{"line_number":402,"context_line":"        fake_net_id \u003d str(uuidutils.generate_uuid())"},{"line_number":403,"context_line":"        self.mock_object(share_types, \u0027get_default_share_type\u0027,"},{"line_number":404,"context_line":"                         mock.Mock(return_value\u003dself.fake_share_type))"}],"source_content_type":"text/x-python","patch_set":9,"id":"bcf9a251_68ccd642","line":401,"range":{"start_line":401,"start_character":23,"end_line":401,"end_character":53},"in_reply_to":"0078ae58_ff63d83b","updated":"2021-01-28 02:50:15.000000000","message":"Done, yes, i will.","commit_id":"26861d28b65b62d0bdcafda061211bacb9051d0d"}],"manila/tests/api/v2/test_share_replicas.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":573,"context_line":"        if method_name in (\u0027index\u0027, \u0027detail\u0027):"},{"line_number":574,"context_line":"            arguments.clear()"},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"        noauthexc \u003d exception.PolicyNotAuthorized(action\u003dstr(method))"},{"line_number":577,"context_line":""},{"line_number":578,"context_line":"        with mock.patch.object("},{"line_number":579,"context_line":"                policy, \u0027check_policy\u0027, mock.Mock(side_effect\u003dnoauthexc)):"}],"source_content_type":"text/x-python","patch_set":8,"id":"36961a4f_7201a265","line":576,"range":{"start_line":576,"start_character":57,"end_line":576,"end_character":68},"updated":"2021-01-27 07:40:18.000000000","message":"method_name","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"c088174068a576005100d4dbf516e95c6ef911dc","unresolved":false,"context_lines":[{"line_number":573,"context_line":"        if method_name in (\u0027index\u0027, \u0027detail\u0027):"},{"line_number":574,"context_line":"            arguments.clear()"},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"        noauthexc \u003d exception.PolicyNotAuthorized(action\u003dstr(method))"},{"line_number":577,"context_line":""},{"line_number":578,"context_line":"        with mock.patch.object("},{"line_number":579,"context_line":"                policy, \u0027check_policy\u0027, mock.Mock(side_effect\u003dnoauthexc)):"}],"source_content_type":"text/x-python","patch_set":8,"id":"24446fdd_c44c33e4","line":576,"range":{"start_line":576,"start_character":57,"end_line":576,"end_character":68},"in_reply_to":"36961a4f_7201a265","updated":"2021-01-27 11:25:22.000000000","message":"Done","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"}],"manila/tests/api/v2/test_share_snapshot_instances.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":192,"context_line":"                \u0027body\u0027: {\u0027FAKE_KEY\u0027: \u0027FAKE_VAL\u0027},"},{"line_number":193,"context_line":"            }"},{"line_number":194,"context_line":""},{"line_number":195,"context_line":"        noauthexc \u003d exception.PolicyNotAuthorized(action\u003dstr(method))"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        with mock.patch.object("},{"line_number":198,"context_line":"                policy, \u0027check_policy\u0027, mock.Mock(side_effect\u003dnoauthexc)):"}],"source_content_type":"text/x-python","patch_set":8,"id":"1908a5ca_382fd67e","line":195,"range":{"start_line":195,"start_character":57,"end_line":195,"end_character":68},"updated":"2021-01-27 07:40:18.000000000","message":"we could just replace this with method_name","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"c088174068a576005100d4dbf516e95c6ef911dc","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                \u0027body\u0027: {\u0027FAKE_KEY\u0027: \u0027FAKE_VAL\u0027},"},{"line_number":193,"context_line":"            }"},{"line_number":194,"context_line":""},{"line_number":195,"context_line":"        noauthexc \u003d exception.PolicyNotAuthorized(action\u003dstr(method))"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        with mock.patch.object("},{"line_number":198,"context_line":"                policy, \u0027check_policy\u0027, mock.Mock(side_effect\u003dnoauthexc)):"}],"source_content_type":"text/x-python","patch_set":8,"id":"4dd3ed64_ae6c7424","line":195,"range":{"start_line":195,"start_character":57,"end_line":195,"end_character":68},"in_reply_to":"1908a5ca_382fd67e","updated":"2021-01-27 11:25:22.000000000","message":"Done","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"}],"manila/tests/cmd/test_manage.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"b8cdcd07b490655d88562b7295b50ae1b7c5b5a7","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        try:"},{"line_number":69,"context_line":"            import bpython"},{"line_number":70,"context_line":"        except ImportError as e:"},{"line_number":71,"context_line":"            self.skipTest(str(e))"},{"line_number":72,"context_line":"        self.mock_object(bpython, \u0027embed\u0027)"},{"line_number":73,"context_line":"        self.shell_commands.run(**kwargs)"},{"line_number":74,"context_line":"        bpython.embed.assert_called_once_with()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_e0112ffa","line":71,"updated":"2020-10-09 10:40:06.000000000","message":"pep8: M325 str() and unicode() cannot be used on an exception.  Remove or use str()","commit_id":"40c7e74eb6ed14f2ecd6cc4b6cd3b5cdbcdd12eb"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"b8cdcd07b490655d88562b7295b50ae1b7c5b5a7","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            import bpython"},{"line_number":79,"context_line":"            import IPython"},{"line_number":80,"context_line":"        except ImportError as e:"},{"line_number":81,"context_line":"            self.skipTest(str(e))"},{"line_number":82,"context_line":"        self.mock_object(bpython, \u0027embed\u0027,"},{"line_number":83,"context_line":"                         mock.Mock(side_effect\u003dImportError()))"},{"line_number":84,"context_line":"        self.mock_object(IPython, \u0027embed\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_40d0db2b","line":81,"updated":"2020-10-09 10:40:06.000000000","message":"pep8: M325 str() and unicode() cannot be used on an exception.  Remove or use str()","commit_id":"40c7e74eb6ed14f2ecd6cc4b6cd3b5cdbcdd12eb"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"b8cdcd07b490655d88562b7295b50ae1b7c5b5a7","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        try:"},{"line_number":92,"context_line":"            import bpython"},{"line_number":93,"context_line":"        except ImportError as e:"},{"line_number":94,"context_line":"            self.skipTest(str(e))"},{"line_number":95,"context_line":"        self.mock_object(bpython, \u0027embed\u0027)"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"        self.shell_commands.run()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_20d5e71b","line":94,"updated":"2020-10-09 10:40:06.000000000","message":"pep8: M325 str() and unicode() cannot be used on an exception.  Remove or use str()","commit_id":"40c7e74eb6ed14f2ecd6cc4b6cd3b5cdbcdd12eb"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"b8cdcd07b490655d88562b7295b50ae1b7c5b5a7","unresolved":false,"context_lines":[{"line_number":102,"context_line":"        try:"},{"line_number":103,"context_line":"            import IPython"},{"line_number":104,"context_line":"        except ImportError as e:"},{"line_number":105,"context_line":"            self.skipTest(str(e))"},{"line_number":106,"context_line":"        self.mock_object(IPython, \u0027embed\u0027)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        self.shell_commands.run(shell\u003d\u0027ipython\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_80d67325","line":105,"updated":"2020-10-09 10:40:06.000000000","message":"pep8: M325 str() and unicode() cannot be used on an exception.  Remove or use str()","commit_id":"40c7e74eb6ed14f2ecd6cc4b6cd3b5cdbcdd12eb"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"b8cdcd07b490655d88562b7295b50ae1b7c5b5a7","unresolved":false,"context_lines":[{"line_number":117,"context_line":"                setattr(IPython.Shell, \u0027IPShell\u0027,"},{"line_number":118,"context_line":"                        mock.Mock(side_effect\u003dImportError()))"},{"line_number":119,"context_line":"        except ImportError as e:"},{"line_number":120,"context_line":"            self.skipTest(str(e))"},{"line_number":121,"context_line":"        self.mock_object(IPython, \u0027embed\u0027,"},{"line_number":122,"context_line":"                         mock.Mock(side_effect\u003dImportError()))"},{"line_number":123,"context_line":"        self.mock_object(readline, \u0027parse_and_bind\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_60db5f4d","line":120,"updated":"2020-10-09 10:40:06.000000000","message":"pep8: M325 str() and unicode() cannot be used on an exception.  Remove or use str()","commit_id":"40c7e74eb6ed14f2ecd6cc4b6cd3b5cdbcdd12eb"}],"manila/tests/network/linux/test_interface.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"b8cdcd07b490655d88562b7295b50ae1b7c5b5a7","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        except TypeError:"},{"line_number":80,"context_line":"            pass"},{"line_number":81,"context_line":"        except Exception as e:"},{"line_number":82,"context_line":"            self.fail(\"Unexpected exception thrown: \u0027%s\u0027\" % str(e))"},{"line_number":83,"context_line":"        else:"},{"line_number":84,"context_line":"            self.fail(\"ExpectedException \u0027TypeError\u0027 not thrown.\")"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_001543e6","line":82,"updated":"2020-10-09 10:40:06.000000000","message":"pep8: M325 str() and unicode() cannot be used on an exception.  Remove or use str()","commit_id":"40c7e74eb6ed14f2ecd6cc4b6cd3b5cdbcdd12eb"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":79,"context_line":"        except TypeError:"},{"line_number":80,"context_line":"            pass"},{"line_number":81,"context_line":"        except Exception as e:"},{"line_number":82,"context_line":"            self.fail(\"Unexpected exception thrown: \u0027%s\u0027\" % str(e))"},{"line_number":83,"context_line":"        else:"},{"line_number":84,"context_line":"            self.fail(\"ExpectedException \u0027TypeError\u0027 not thrown.\")"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"ee38a6cc_51186143","line":82,"range":{"start_line":82,"start_character":59,"end_line":82,"end_character":66},"updated":"2021-01-27 07:40:18.000000000","message":"%s is already formatting this to a string, explicit casting isn\u0027t necessary","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"c088174068a576005100d4dbf516e95c6ef911dc","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        except TypeError:"},{"line_number":80,"context_line":"            pass"},{"line_number":81,"context_line":"        except Exception as e:"},{"line_number":82,"context_line":"            self.fail(\"Unexpected exception thrown: \u0027%s\u0027\" % str(e))"},{"line_number":83,"context_line":"        else:"},{"line_number":84,"context_line":"            self.fail(\"ExpectedException \u0027TypeError\u0027 not thrown.\")"},{"line_number":85,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"5dec0996_8ce35f06","line":82,"range":{"start_line":82,"start_character":59,"end_line":82,"end_character":66},"in_reply_to":"ee38a6cc_51186143","updated":"2021-01-27 11:25:22.000000000","message":"Done","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"}],"manila/tests/test_utils.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":805,"context_line":""},{"line_number":806,"context_line":"    def test_convert_str_str_input(self):"},{"line_number":807,"context_line":"        self.mock_object(encodeutils, \u0027safe_encode\u0027)"},{"line_number":808,"context_line":"        input_value \u003d str(\"string_input\")"},{"line_number":809,"context_line":""},{"line_number":810,"context_line":"        output_value \u003d utils.convert_str(input_value)"},{"line_number":811,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"43116b4a_0544956e","line":808,"range":{"start_line":808,"start_character":22,"end_line":808,"end_character":25},"updated":"2021-01-27 07:40:18.000000000","message":"it\u0027s already a string","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"c088174068a576005100d4dbf516e95c6ef911dc","unresolved":false,"context_lines":[{"line_number":805,"context_line":""},{"line_number":806,"context_line":"    def test_convert_str_str_input(self):"},{"line_number":807,"context_line":"        self.mock_object(encodeutils, \u0027safe_encode\u0027)"},{"line_number":808,"context_line":"        input_value \u003d str(\"string_input\")"},{"line_number":809,"context_line":""},{"line_number":810,"context_line":"        output_value \u003d utils.convert_str(input_value)"},{"line_number":811,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"3bd0c20a_74e5dd1c","line":808,"range":{"start_line":808,"start_character":22,"end_line":808,"end_character":25},"in_reply_to":"43116b4a_0544956e","updated":"2021-01-27 11:25:22.000000000","message":"Done","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"}],"tox.ini":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e6be1575ed206d4184f908157278ae6a99ff70e0","unresolved":true,"context_lines":[{"line_number":134,"context_line":"# W503 line break before binary operator"},{"line_number":135,"context_line":"# W504 line break after binary operator"},{"line_number":136,"context_line":"# M325 str() can be used on an exception in PY3."},{"line_number":137,"context_line":"ignore \u003d E123,E402,W503,W504,M325"},{"line_number":138,"context_line":"builtins \u003d _"},{"line_number":139,"context_line":"# [H106] Don\u0027t put vim configuration in source files."},{"line_number":140,"context_line":"# [H203] Use assertIs(Not)None to check for None."}],"source_content_type":"text/x-properties","patch_set":8,"id":"c7590208_7dd5b94a","line":137,"range":{"start_line":137,"start_character":29,"end_line":137,"end_character":33},"updated":"2021-01-27 07:40:18.000000000","message":"This rule is gone, so it can be removed from here","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"c088174068a576005100d4dbf516e95c6ef911dc","unresolved":false,"context_lines":[{"line_number":134,"context_line":"# W503 line break before binary operator"},{"line_number":135,"context_line":"# W504 line break after binary operator"},{"line_number":136,"context_line":"# M325 str() can be used on an exception in PY3."},{"line_number":137,"context_line":"ignore \u003d E123,E402,W503,W504,M325"},{"line_number":138,"context_line":"builtins \u003d _"},{"line_number":139,"context_line":"# [H106] Don\u0027t put vim configuration in source files."},{"line_number":140,"context_line":"# [H203] Use assertIs(Not)None to check for None."}],"source_content_type":"text/x-properties","patch_set":8,"id":"015ebe48_08fb0744","line":137,"range":{"start_line":137,"start_character":29,"end_line":137,"end_character":33},"in_reply_to":"c7590208_7dd5b94a","updated":"2021-01-27 11:25:22.000000000","message":"Done","commit_id":"da8286768188ba077928be739df1820bb74cf8e6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b565b510fc39e18f76fb4ce51b4102b4a614677d","unresolved":true,"context_lines":[{"line_number":133,"context_line":"# E402 module level import not at top of file"},{"line_number":134,"context_line":"# W503 line break before binary operator"},{"line_number":135,"context_line":"# W504 line break after binary operator"},{"line_number":136,"context_line":"# M325 str() can be used on an exception in PY3."},{"line_number":137,"context_line":"ignore \u003d E123,E402,W503,W504"},{"line_number":138,"context_line":"builtins \u003d _"},{"line_number":139,"context_line":"# [H106] Don\u0027t put vim configuration in source files."}],"source_content_type":"text/x-properties","patch_set":9,"id":"388f17f3_53fa286a","line":136,"range":{"start_line":136,"start_character":0,"end_line":136,"end_character":48},"updated":"2021-01-27 18:27:42.000000000","message":"this line isn\u0027t necessary - M325 doesn\u0027t exist anymore - you have deleted it, lets remove all references to it so it doesn\u0027t end up confusing people in the future.","commit_id":"26861d28b65b62d0bdcafda061211bacb9051d0d"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"430b16741aee02c4344b9ea732a1c4ce1da9208f","unresolved":true,"context_lines":[{"line_number":133,"context_line":"# E402 module level import not at top of file"},{"line_number":134,"context_line":"# W503 line break before binary operator"},{"line_number":135,"context_line":"# W504 line break after binary operator"},{"line_number":136,"context_line":"# M325 str() can be used on an exception in PY3."},{"line_number":137,"context_line":"ignore \u003d E123,E402,W503,W504"},{"line_number":138,"context_line":"builtins \u003d _"},{"line_number":139,"context_line":"# [H106] Don\u0027t put vim configuration in source files."}],"source_content_type":"text/x-properties","patch_set":9,"id":"ab1f9df7_5ebb1e97","line":136,"range":{"start_line":136,"start_character":0,"end_line":136,"end_character":48},"in_reply_to":"388f17f3_53fa286a","updated":"2021-01-28 02:50:15.000000000","message":"sorry, i missed it.","commit_id":"26861d28b65b62d0bdcafda061211bacb9051d0d"}]}