)]}'
{"manila/share_group/__init__.py":[{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"5b996fd8d9578cd50775d31b325a2ab35c88a70a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1a6eadb0_d2a5e5b5","updated":"2016-12-12 18:11:02.000000000","message":"For the diff it is unfortunate that this was copied from tests/cg instead of cg, but no harm copying around an empty file.","commit_id":"3d2610a073f607302cbabc5daccb786ee94da1ab"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"4a418edfc78ec5675ffa9dd17e360f303db95b92","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1a6eadb0_ca1d0a11","in_reply_to":"1a6eadb0_d2a5e5b5","updated":"2016-12-13 11:59:48.000000000","message":"It was marked as \"copied\" because I haven\u0027t deleted original __init__.py file and its directory. Deleted, reuploaded commit. Now it is nice all \"R\"s \u003d)","commit_id":"3d2610a073f607302cbabc5daccb786ee94da1ab"}],"manila/share_group/api.py":[{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"3ffcce70c429d641f9f77eb1e98156dbd679c18b","unresolved":false,"context_lines":[{"line_number":50,"context_line":"    def create(self, context, name\u003dNone, description\u003dNone,"},{"line_number":51,"context_line":"               share_type_ids\u003dNone, source_cgsnapshot_id\u003dNone,"},{"line_number":52,"context_line":"               share_network_id\u003dNone):"},{"line_number":53,"context_line":"        \"\"\"Create new consistency group.\"\"\""},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        cgsnapshot \u003d None"},{"line_number":56,"context_line":"        original_cg \u003d None"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a6eadb0_d43771e1","line":53,"range":{"start_line":53,"start_character":22,"end_line":53,"end_character":33},"updated":"2016-12-13 10:55:16.000000000","message":"I would expect all strings, comments docstrings, method names, variables, etc to be changed in a separate commit as well (like this one), this would make the main patch much easier to read wrt to what is logic changes and what is renaming","commit_id":"3d2610a073f607302cbabc5daccb786ee94da1ab"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"3ffcce70c429d641f9f77eb1e98156dbd679c18b","unresolved":false,"context_lines":[{"line_number":52,"context_line":"               share_network_id\u003dNone):"},{"line_number":53,"context_line":"        \"\"\"Create new consistency group.\"\"\""},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        cgsnapshot \u003d None"},{"line_number":56,"context_line":"        original_cg \u003d None"},{"line_number":57,"context_line":"        # NOTE(gouthamr): share_server_id is inherited from the parent CG if a"},{"line_number":58,"context_line":"        # CG snapshot is specified, else, it will be set in the share manager."},{"line_number":59,"context_line":"        share_server_id \u003d None"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a6eadb0_f4346ddf","line":56,"range":{"start_line":55,"start_character":8,"end_line":56,"end_character":26},"updated":"2016-12-13 10:55:16.000000000","message":"variables","commit_id":"3d2610a073f607302cbabc5daccb786ee94da1ab"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"3ffcce70c429d641f9f77eb1e98156dbd679c18b","unresolved":false,"context_lines":[{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        cgsnapshot \u003d None"},{"line_number":56,"context_line":"        original_cg \u003d None"},{"line_number":57,"context_line":"        # NOTE(gouthamr): share_server_id is inherited from the parent CG if a"},{"line_number":58,"context_line":"        # CG snapshot is specified, else, it will be set in the share manager."},{"line_number":59,"context_line":"        share_server_id \u003d None"},{"line_number":60,"context_line":"        if source_cgsnapshot_id:"},{"line_number":61,"context_line":"            cgsnapshot \u003d self.db.cgsnapshot_get(context, source_cgsnapshot_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a6eadb0_74577d02","line":58,"range":{"start_line":57,"start_character":71,"end_line":58,"end_character":21},"updated":"2016-12-13 10:55:16.000000000","message":"comments","commit_id":"3d2610a073f607302cbabc5daccb786ee94da1ab"}]}