)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"114a3006176b6bd6d346e2a7284040b239e81dc8","unresolved":false,"context_lines":[{"line_number":18,"context_line":"Convert to list instead, and skip conversion for `collections.Sequence`"},{"line_number":19,"context_line":"objects only that support all the methods used for this object"},{"line_number":20,"context_line":"further in this method."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I141a28d96f71c06a1ebe44d7067ccf4609e22db6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1fcbff3d_4e2af3e7","line":21,"updated":"2019-04-20 12:39:53.000000000","message":"Could you file a bug on this?","commit_id":"956ec38a377f3315dc2859635c8966aba4af5df3"}],"openstack_dashboard/api/neutron.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"93c1a931c8f41fbf2fb077e6ec116fadb36e3076","unresolved":false,"context_lines":[{"line_number":852,"context_line":"        # filter_values) and do not consider other filter conditions"},{"line_number":853,"context_line":"        # which may be specified in **params."},{"line_number":854,"context_line":"        if not isinstance(filter_values, (list, tuple, set, frozenset)):"},{"line_number":855,"context_line":"            filter_values \u003d [filter_values]"},{"line_number":856,"context_line":""},{"line_number":857,"context_line":"        # Length of each query filter is:"},{"line_number":858,"context_line":"        # \u003ckey\u003e\u003d\u003cvalue\u003e\u0026 (e.g., id\u003d\u003cuuid\u003e)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fce034c_34e5d409","side":"PARENT","line":855,"updated":"2019-04-19 07:33:53.000000000","message":"This if clause itself looks unnecessary. The original intention of this if clause is to accept non-iterable filter values like a  string (as neutronclient python binding accepts such values as filter_values). However, there is no such consumer now.\n\nThe if clause itself is very similar to the proposed one, but the meaning is completely different. How about dropping this first and adding the new one in a separate patch?","commit_id":"c99cabf19f4de85d6b2d09d7d301240450b365d5"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"93c1a931c8f41fbf2fb077e6ec116fadb36e3076","unresolved":false,"context_lines":[{"line_number":850,"context_line":""},{"line_number":851,"context_line":"        # We consider only the filter condition from (filter_attr,"},{"line_number":852,"context_line":"        # filter_values) and do not consider other filter conditions"},{"line_number":853,"context_line":"        # which may be specified in **params."},{"line_number":854,"context_line":"        # NOTE(pas-ha) this will produce a deprecation warning in Py37"},{"line_number":855,"context_line":"        # and will not work in Py38, and six.moves also does not support it"},{"line_number":856,"context_line":"        # (see https://github.com/benjaminp/six/issues/155)."}],"source_content_type":"text/x-python","patch_set":2,"id":"3fce034c_54dae04a","line":853,"updated":"2019-04-19 07:33:53.000000000","message":"Could you add a blank line here?\n\nThe above comment and the comment you added are not related at all.","commit_id":"956ec38a377f3315dc2859635c8966aba4af5df3"}]}