)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"29d70ba68a0cdd12990f7ed4500e70bbedbc5e0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f87efa88_8c8f0083","updated":"2022-10-10 15:22:24.000000000","message":"Looks good, but we need a test for the list situation that you added in designate/sqlalchemy/base.py#L132","commit_id":"9899c12d2082e1178344b3229935f3824884c535"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"10255403b389be89bf9969d8b01be8f422b5ba2b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d94b2b42_2cedc229","updated":"2022-10-11 14:07:02.000000000","message":"LGTM - thanks tons for expanding the test coverage to the lists and more.\n\n{0} designate.tests.test_sqlalchemy.SQLAlchemyTestCase.test_list [2.143314s] ... ok","commit_id":"0ca13a84e380b8799d310e7262e464031aef96e7"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"455ee5af1961acf230f6df87bafc853ce0cd37e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"407a9858_3cc2dba8","updated":"2022-10-24 14:40:07.000000000","message":"LGTM - thanks for the unicode and consistency of double verses single quots ","commit_id":"f31dc7d9f94c0780b306efccec4f612249172c0c"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"366e5606a91023a232a41390dd73d04b2454806d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"418126f1_0ee7d490","updated":"2022-11-18 22:13:41.000000000","message":"LGTM","commit_id":"b48bda4267053621f31dbcfe425b9ae0ab9f3008"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"26a6d98de76674c077b82a170de79c6eaa0bd55a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c7bd7d6d_13a6b660","updated":"2022-11-19 04:13:41.000000000","message":"recheck","commit_id":"b48bda4267053621f31dbcfe425b9ae0ab9f3008"}],"designate/sqlalchemy/base.py":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"29d70ba68a0cdd12990f7ed4500e70bbedbc5e0e","unresolved":true,"context_lines":[{"line_number":129,"context_line":"                    )"},{"line_number":130,"context_line":"                else:"},{"line_number":131,"context_line":"                    query \u003d query.where(column \u003d\u003d value)"},{"line_number":132,"context_line":"            elif isinstance(value, list):"},{"line_number":133,"context_line":"                query \u003d query.where(column.in_(value))"},{"line_number":134,"context_line":"            else:"},{"line_number":135,"context_line":"                query \u003d query.where(column \u003d\u003d value)"}],"source_content_type":"text/x-python","patch_set":1,"id":"577a0e02_87c9f7b9","line":132,"updated":"2022-10-10 15:22:24.000000000","message":"Need a test for this in designate/tests/test_sqlalchemy.py","commit_id":"9899c12d2082e1178344b3229935f3824884c535"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"77c6103037a90bce9c334213a42774e8c130a27f","unresolved":false,"context_lines":[{"line_number":129,"context_line":"                    )"},{"line_number":130,"context_line":"                else:"},{"line_number":131,"context_line":"                    query \u003d query.where(column \u003d\u003d value)"},{"line_number":132,"context_line":"            elif isinstance(value, list):"},{"line_number":133,"context_line":"                query \u003d query.where(column.in_(value))"},{"line_number":134,"context_line":"            else:"},{"line_number":135,"context_line":"                query \u003d query.where(column \u003d\u003d value)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a8493b8f_60974ef8","line":132,"in_reply_to":"248f1f63_5f9c300a","updated":"2022-10-11 02:29:02.000000000","message":"Fixed.","commit_id":"9899c12d2082e1178344b3229935f3824884c535"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"10255403b389be89bf9969d8b01be8f422b5ba2b","unresolved":true,"context_lines":[{"line_number":129,"context_line":"                    )"},{"line_number":130,"context_line":"                else:"},{"line_number":131,"context_line":"                    query \u003d query.where(column \u003d\u003d value)"},{"line_number":132,"context_line":"            elif isinstance(value, list):"},{"line_number":133,"context_line":"                query \u003d query.where(column.in_(value))"},{"line_number":134,"context_line":"            else:"},{"line_number":135,"context_line":"                query \u003d query.where(column \u003d\u003d value)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3b2bf072_c1429a64","line":132,"in_reply_to":"248f1f63_5f9c300a","updated":"2022-10-11 14:07:02.000000000","message":"Yes, a test for list, yes I understand it didn\u0027t change","commit_id":"9899c12d2082e1178344b3229935f3824884c535"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"db73f080f7494515af79fc5e3ac2e5518aa84a26","unresolved":false,"context_lines":[{"line_number":129,"context_line":"                    )"},{"line_number":130,"context_line":"                else:"},{"line_number":131,"context_line":"                    query \u003d query.where(column \u003d\u003d value)"},{"line_number":132,"context_line":"            elif isinstance(value, list):"},{"line_number":133,"context_line":"                query \u003d query.where(column.in_(value))"},{"line_number":134,"context_line":"            else:"},{"line_number":135,"context_line":"                query \u003d query.where(column \u003d\u003d value)"}],"source_content_type":"text/x-python","patch_set":1,"id":"54ba3048_2abc63c4","line":132,"in_reply_to":"3b2bf072_c1429a64","updated":"2022-10-12 17:31:41.000000000","message":"Ack","commit_id":"9899c12d2082e1178344b3229935f3824884c535"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"b6e99b53788712c3774a7ab43bba7d7456d3315e","unresolved":true,"context_lines":[{"line_number":129,"context_line":"                    )"},{"line_number":130,"context_line":"                else:"},{"line_number":131,"context_line":"                    query \u003d query.where(column \u003d\u003d value)"},{"line_number":132,"context_line":"            elif isinstance(value, list):"},{"line_number":133,"context_line":"                query \u003d query.where(column.in_(value))"},{"line_number":134,"context_line":"            else:"},{"line_number":135,"context_line":"                query \u003d query.where(column \u003d\u003d value)"}],"source_content_type":"text/x-python","patch_set":1,"id":"248f1f63_5f9c300a","line":132,"in_reply_to":"577a0e02_87c9f7b9","updated":"2022-10-10 17:01:14.000000000","message":"More testing is always good. Were you referring to this line in particular or the whole function? Because this is one of the few lines in this function that didn\u0027t change.","commit_id":"9899c12d2082e1178344b3229935f3824884c535"}]}