)]}'
{"openstackclient/object/v1/container.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"595a1ea89096adebd38dc8a720439c502bf0540f","unresolved":false,"context_lines":[{"line_number":53,"context_line":"                container,"},{"line_number":54,"context_line":"            )"},{"line_number":55,"context_line":"            results.append(data)"},{"line_number":56,"context_line":"            columns \u003d data.keys()"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        return (columns,"},{"line_number":59,"context_line":"                (utils.get_dict_properties("}],"source_content_type":"text/x-python","patch_set":1,"id":"7ac7a964_59f86a7c","line":56,"updated":"2014-08-15 03:17:56.000000000","message":"same comment as in object","commit_id":"d7b9e46c764b9099bca269092dd089303fd4d444"}],"openstackclient/object/v1/object.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"595a1ea89096adebd38dc8a720439c502bf0540f","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            data.pop(\u0027account\u0027, None)"},{"line_number":62,"context_line":"            data.pop(\u0027x-trans-id\u0027, None)"},{"line_number":63,"context_line":"            results.append(data)"},{"line_number":64,"context_line":"            columns \u003d data.keys()"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        return (columns,"},{"line_number":67,"context_line":"                (utils.get_dict_properties("}],"source_content_type":"text/x-python","patch_set":1,"id":"7ac7a964_39048e7a","line":64,"updated":"2014-08-15 03:17:56.000000000","message":"columns is just going to be the last objects keys, seems wrong to have it in the loop","commit_id":"d7b9e46c764b9099bca269092dd089303fd4d444"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"20bec40b75cecbc936facecd211d7aaf2754ff90","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            results.append(data)"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        if results:"},{"line_number":66,"context_line":"            columns \u003d results[0].keys()"},{"line_number":67,"context_line":"        return (columns,"},{"line_number":68,"context_line":"                (utils.get_dict_properties("},{"line_number":69,"context_line":"                    s, columns,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7ac7a964_75e9947f","line":66,"updated":"2014-08-15 15:19:22.000000000","message":"mehhh, still not great, but i\u0027m not sure how else to do this. (aside from just hard coding it)","commit_id":"c15a7c48a95295c1acde1dac24af3f547665971d"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"d3f328fe84cbefcc8f3e2ed5994f23422896a41e","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            results.append(data)"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        if results:"},{"line_number":66,"context_line":"            columns \u003d results[0].keys()"},{"line_number":67,"context_line":"        return (columns,"},{"line_number":68,"context_line":"                (utils.get_dict_properties("},{"line_number":69,"context_line":"                    s, columns,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7ac7a964_5b13e965","line":66,"in_reply_to":"7ac7a964_75e9947f","updated":"2014-08-15 16:09:28.000000000","message":"hard coding is fine, that would save me popping \u0027account\u0027 and \u0027x-trans-id\u0027  since they cluttered up the output.","commit_id":"c15a7c48a95295c1acde1dac24af3f547665971d"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"20bec40b75cecbc936facecd211d7aaf2754ff90","unresolved":false,"context_lines":[{"line_number":87,"context_line":"            \u0027objects\u0027,"},{"line_number":88,"context_line":"            metavar\u003d\u0027\u003cobject-name\u003e\u0027,"},{"line_number":89,"context_line":"            nargs\u003d\"+\","},{"line_number":90,"context_line":"            help\u003d\u0027Object(s) to delete\u0027,"},{"line_number":91,"context_line":"        )"},{"line_number":92,"context_line":"        return parser"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7ac7a964_552f38f6","line":90,"updated":"2014-08-15 15:19:22.000000000","message":"in delete, we expect:\n\n  os object delete \u003ccontainer\u003e \u003cobj1\u003e \u003cobj2\u003e ... \u003cobjN\u003e\n\nwhich is the opposite of create:\n\n  os object create \u003cobj1\u003e \u003cobj2\u003e ... \u003cobjN\u003e \u003ccontainer\u003e \n\nWe should be consistent","commit_id":"c15a7c48a95295c1acde1dac24af3f547665971d"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"d3f328fe84cbefcc8f3e2ed5994f23422896a41e","unresolved":false,"context_lines":[{"line_number":87,"context_line":"            \u0027objects\u0027,"},{"line_number":88,"context_line":"            metavar\u003d\u0027\u003cobject-name\u003e\u0027,"},{"line_number":89,"context_line":"            nargs\u003d\"+\","},{"line_number":90,"context_line":"            help\u003d\u0027Object(s) to delete\u0027,"},{"line_number":91,"context_line":"        )"},{"line_number":92,"context_line":"        return parser"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7ac7a964_469f8cb4","line":90,"in_reply_to":"7ac7a964_552f38f6","updated":"2014-08-15 16:09:28.000000000","message":"Yeh, awkward.  Maybe:\n\n1) If the verb was upload instead of create, there wouldn\u0027t be as much need to have consistency.\n2) If the verb was copy, we could have the final argument as destination instead and maybe a --download \u003csource_container\u003e option for download.","commit_id":"c15a7c48a95295c1acde1dac24af3f547665971d"}]}