)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"ee6db0dc17a71abf637e0f4da6b24ea74d1f8bf9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[Trivial fix]Remove unnecessary slash"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is to remove the unncessary slash when the line doesn\u0027t"},{"line_number":10,"context_line":"exceed the length of 79, to make the code more readable."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I989507020d5bb7f216d374575cebe005d27225da"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"dfbec78f_87792b30","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":56},"updated":"2019-05-10 08:22:34.000000000","message":"This is inaccurate now, but thanks for the expanding the patch to remove all those backslashes!","commit_id":"9078043b0451eafabc7a104dee25ee50a5bb5a23"}],"networking_ovn/octavia/ovn_driver.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"16bc67cb343255f8555c9e62786e974b3105da0c","unresolved":false,"context_lines":[{"line_number":1173,"context_line":"                    status[\u0027members\u0027][0][\u0027operating_status\u0027] \u003d constants.ONLINE"},{"line_number":1174,"context_line":"                else:"},{"line_number":1175,"context_line":"                    self._remove_member(member, ovn_lb, pool_key)"},{"line_number":1176,"context_line":"                    status[\u0027members\u0027][0][\u0027operating_status\u0027] \u003d \\"},{"line_number":1177,"context_line":"                        constants.OFFLINE"},{"line_number":1178,"context_line":""},{"line_number":1179,"context_line":"            pool_listeners \u003d self._get_pool_listeners(ovn_lb, pool_key)"}],"source_content_type":"text/x-python","patch_set":1,"id":"dfbec78f_c69e8106","line":1176,"range":{"start_line":1176,"start_character":62,"end_line":1176,"end_character":64},"updated":"2019-05-07 12:02:46.000000000","message":"Ideally we should replace all \\ with () for joining lines, the PEP8 guide states:\n\n\"The preferred way of wrapping long lines is by using Python\u0027s implied line continuation inside parentheses, brackets and braces. Long lines can be broken over multiple lines by wrapping expressions in parentheses. These should be used in preference to using a backslash for line continuation.\"\n\nlink: https://www.python.org/dev/peps/pep-0008/#maximum-line-length\n\n...\n\n\nTho, this is somewhat different than the patch is proposing. But maybe you may reconsider changing it as well.","commit_id":"970e391d30431dee5340a40be25f705a0e22cc09"},{"author":{"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"},"change_message_id":"4ad42dfa819182f4cc1dddf2b86faa62d7cffa88","unresolved":false,"context_lines":[{"line_number":1173,"context_line":"                    status[\u0027members\u0027][0][\u0027operating_status\u0027] \u003d constants.ONLINE"},{"line_number":1174,"context_line":"                else:"},{"line_number":1175,"context_line":"                    self._remove_member(member, ovn_lb, pool_key)"},{"line_number":1176,"context_line":"                    status[\u0027members\u0027][0][\u0027operating_status\u0027] \u003d \\"},{"line_number":1177,"context_line":"                        constants.OFFLINE"},{"line_number":1178,"context_line":""},{"line_number":1179,"context_line":"            pool_listeners \u003d self._get_pool_listeners(ovn_lb, pool_key)"}],"source_content_type":"text/x-python","patch_set":1,"id":"dfbec78f_62376eb2","line":1176,"range":{"start_line":1176,"start_character":62,"end_line":1176,"end_character":64},"in_reply_to":"dfbec78f_c69e8106","updated":"2019-05-08 09:51:29.000000000","message":"+1","commit_id":"970e391d30431dee5340a40be25f705a0e22cc09"}]}