)]}'
{"novaclient/tests/v1_1/contrib/test_cells.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"16f32d9e97bfe8c36ca35cf8a98e811860f62e2a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":"from novaclient import extension"},{"line_number":16,"context_line":"from novaclient.v1_1.contrib import cells"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from novaclient.tests import utils"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2F2hg%3D","line":15,"updated":"2013-06-07 15:12:03.000000000","message":"2 nits here.  The first is a femtonit: I like to have a blank line between the license header and the first import.  The second is a hacking nit, which is that all imports from the project (here, novaclient) need to be grouped together and in alphabetical order; so, this list of imports should be:\n\n    from novaclient import extension\n    from novaclient.tests import utils\n    from novaclient.tests.v1_1.contrib import fakes\n    from novaclient.v1_1.contrib import cells","commit_id":"9ac52570a4ebf1541baf8df994d3a8a19d39deea"}],"novaclient/v1_1/contrib/cells.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"16f32d9e97bfe8c36ca35cf8a98e811860f62e2a","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        metavar\u003d\u0027\u003ccell-name\u003e\u0027,"},{"line_number":40,"context_line":"        help\u003d\u0027show cell.\u0027)"},{"line_number":41,"context_line":"def do_cell_show(cs, args):"},{"line_number":42,"context_line":"    \"\"\"Show an action.\"\"\""},{"line_number":43,"context_line":"    cell \u003d cs.cells.get(args.cell)"},{"line_number":44,"context_line":"    utils.print_dict(cell._info)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2F2hE%3D","line":42,"updated":"2013-06-07 15:12:03.000000000","message":"This docstring does not explain the purpose of this new command.  Remember, do_* function docstrings are used as the help text for the command.","commit_id":"9ac52570a4ebf1541baf8df994d3a8a19d39deea"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"f0d0ff89c0a993816564835532c3e91a151a7928","unresolved":false,"context_lines":[{"line_number":57,"context_line":"@utils.arg(\u0027--cell\u0027,"},{"line_number":58,"context_line":"        metavar\u003d\u0027\u003ccell-name\u003e\u0027,"},{"line_number":59,"context_line":"        help\u003d\u0027show cell-capacities.\u0027,"},{"line_number":60,"context_line":"        default\u003dNone)"},{"line_number":61,"context_line":"def do_cell_capacities(cs, args):"},{"line_number":62,"context_line":"    \"\"\"Get cell capacities for a given cell or all cells if not given.\"\"\""},{"line_number":63,"context_line":"    cell \u003d cs.cells.capacities(args.cell)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPX%2F%2FyuA%3D","line":60,"updated":"2013-06-10 15:37:27.000000000","message":"Please add help text to this argument.  You should be able to explain that the capacities for all cells will be given if this argument is omitted, thus cleaning up the docstring.","commit_id":"2acf0f001264c87b9de2fa442f6aca6889727a26"},{"author":{"_account_id":6992,"name":"Kaushik Chandrashekar","email":"kaushik.chandrashekar@rackspace.com","username":"kaushikc"},"change_message_id":"d9f285b9f1c83ff693ed57d57426d5dccd4ef5c5","unresolved":false,"context_lines":[{"line_number":57,"context_line":"@utils.arg(\u0027--cell\u0027,"},{"line_number":58,"context_line":"        metavar\u003d\u0027\u003ccell-name\u003e\u0027,"},{"line_number":59,"context_line":"        help\u003d\u0027show cell-capacities.\u0027,"},{"line_number":60,"context_line":"        default\u003dNone)"},{"line_number":61,"context_line":"def do_cell_capacities(cs, args):"},{"line_number":62,"context_line":"    \"\"\"Get cell capacities for a given cell or all cells if not given.\"\"\""},{"line_number":63,"context_line":"    cell \u003d cs.cells.capacities(args.cell)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPX%2F%2FyqU%3D","line":60,"in_reply_to":"AAAAPX%2F%2FyuA%3D","updated":"2013-06-10 16:00:40.000000000","message":"Done","commit_id":"2acf0f001264c87b9de2fa442f6aca6889727a26"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"f0d0ff89c0a993816564835532c3e91a151a7928","unresolved":false,"context_lines":[{"line_number":61,"context_line":"def do_cell_capacities(cs, args):"},{"line_number":62,"context_line":"    \"\"\"Get cell capacities for a given cell or all cells if not given.\"\"\""},{"line_number":63,"context_line":"    cell \u003d cs.cells.capacities(args.cell)"},{"line_number":64,"context_line":"    print \"Ram Available: %s MB\" % cell.capacities[\u0027ram_free\u0027][\u0027total_mb\u0027]"},{"line_number":65,"context_line":"    utils.print_dict(cell.capacities[\u0027ram_free\u0027][\u0027units_by_mb\u0027],"},{"line_number":66,"context_line":"                     dict_property\u003d\u0027Ram(MB)\u0027, dict_value\u003d\"Units\")"},{"line_number":67,"context_line":"    print \"\\nDisk Available: %s MB\" % cell.capacities[\u0027disk_free\u0027][\u0027total_mb\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPX%2F%2Fytc%3D","line":64,"updated":"2013-06-10 15:37:27.000000000","message":"Let\u0027s please use the print function style here, to help maintain Python 3 compatibility.  Basically all this means is putting parentheses around the argument to the \"print\" statement, since we\u0027re not making use of the other print() function features.","commit_id":"2acf0f001264c87b9de2fa442f6aca6889727a26"},{"author":{"_account_id":6992,"name":"Kaushik Chandrashekar","email":"kaushik.chandrashekar@rackspace.com","username":"kaushikc"},"change_message_id":"d9f285b9f1c83ff693ed57d57426d5dccd4ef5c5","unresolved":false,"context_lines":[{"line_number":61,"context_line":"def do_cell_capacities(cs, args):"},{"line_number":62,"context_line":"    \"\"\"Get cell capacities for a given cell or all cells if not given.\"\"\""},{"line_number":63,"context_line":"    cell \u003d cs.cells.capacities(args.cell)"},{"line_number":64,"context_line":"    print \"Ram Available: %s MB\" % cell.capacities[\u0027ram_free\u0027][\u0027total_mb\u0027]"},{"line_number":65,"context_line":"    utils.print_dict(cell.capacities[\u0027ram_free\u0027][\u0027units_by_mb\u0027],"},{"line_number":66,"context_line":"                     dict_property\u003d\u0027Ram(MB)\u0027, dict_value\u003d\"Units\")"},{"line_number":67,"context_line":"    print \"\\nDisk Available: %s MB\" % cell.capacities[\u0027disk_free\u0027][\u0027total_mb\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPX%2F%2FyqQ%3D","line":64,"in_reply_to":"AAAAPX%2F%2Fytc%3D","updated":"2013-06-10 16:00:40.000000000","message":"Done","commit_id":"2acf0f001264c87b9de2fa442f6aca6889727a26"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"bca0cec14897361401a5e6801960655b630a182e","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"@utils.arg(\u0027cell\u0027,"},{"line_number":49,"context_line":"        metavar\u003d\u0027\u003ccell-name\u003e\u0027,"},{"line_number":50,"context_line":"        help\u003d\u0027show cell.\u0027)"},{"line_number":51,"context_line":"def do_cell_show(cs, args):"},{"line_number":52,"context_line":"    \"\"\"Show details of a given cell.\"\"\""},{"line_number":53,"context_line":"    cell \u003d cs.cells.get(args.cell)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2FwjE%3D","line":50,"updated":"2013-06-11 08:26:02.000000000","message":"Maybe want some better help here too? At least \"Show\" not \"show\"","commit_id":"f037cb16060012ced8ec325221ab85bcf067a40a"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"926d94945dca8eb7fbf7a26da7e350e0216e1a4e","unresolved":false,"context_lines":[{"line_number":60,"context_line":"                \"capacities of a specific cell\","},{"line_number":61,"context_line":"        default\u003dNone)"},{"line_number":62,"context_line":"def do_cell_capacities(cs, args):"},{"line_number":63,"context_line":"    \"\"\"Get cell capacities for a given cell or all cells if not given.\"\"\""},{"line_number":64,"context_line":"    cell \u003d cs.cells.capacities(args.cell)"},{"line_number":65,"context_line":"    print(\"Ram Available: %s MB\" % cell.capacities[\u0027ram_free\u0027][\u0027total_mb\u0027])"},{"line_number":66,"context_line":"    utils.print_dict(cell.capacities[\u0027ram_free\u0027][\u0027units_by_mb\u0027],"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPX%2F%2FyZQ%3D","line":63,"updated":"2013-06-10 17:32:20.000000000","message":"femtonit: Now that you\u0027ve added the help string to --cell, this can be reworded to be a bit less awkward.","commit_id":"f037cb16060012ced8ec325221ab85bcf067a40a"}]}