)]}'
{"examples/list.py":[{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"5c6ca6c210f4b52777665d639f0402a40eb2f0a4","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    args \u003d argument.rpartition(\u0027.\u0027)"},{"line_number":22,"context_line":"    from_str \u003d args[0]"},{"line_number":23,"context_line":"    class_str \u003d args[2]"},{"line_number":24,"context_line":"    mod \u003d __import__(from_str, fromlist\u003d[class_str])"},{"line_number":25,"context_line":"    return getattr(mod, class_str)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_f6008f68","line":24,"updated":"2014-06-23 16:55:15.000000000","message":"Sadly this way of writing this is frought with complications, the right way to do this is:\n\n    __import__(from_str)\n    mod \u003d sys.modules[from_str]","commit_id":"bbe0be6562520cd6dc3dc798e1ef2c1ecd2cbb3b"},{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"5c6ca6c210f4b52777665d639f0402a40eb2f0a4","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"def run_list(opts):"},{"line_number":29,"context_line":"    sess \u003d session.make_session(opts)"},{"line_number":30,"context_line":"    klazz \u003d make_resource(opts)"},{"line_number":31,"context_line":"    for objay in klazz.list(sess):"},{"line_number":32,"context_line":"        print(str(objay))"},{"line_number":33,"context_line":"    return"}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_36cbd74b","line":30,"updated":"2014-06-23 16:55:15.000000000","message":"`cls` or `klass` is way more common in Python.","commit_id":"bbe0be6562520cd6dc3dc798e1ef2c1ecd2cbb3b"},{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"5c6ca6c210f4b52777665d639f0402a40eb2f0a4","unresolved":false,"context_lines":[{"line_number":28,"context_line":"def run_list(opts):"},{"line_number":29,"context_line":"    sess \u003d session.make_session(opts)"},{"line_number":30,"context_line":"    klazz \u003d make_resource(opts)"},{"line_number":31,"context_line":"    for objay in klazz.list(sess):"},{"line_number":32,"context_line":"        print(str(objay))"},{"line_number":33,"context_line":"    return"},{"line_number":34,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_b62d2700","line":31,"updated":"2014-06-23 16:55:15.000000000","message":"`obj`","commit_id":"bbe0be6562520cd6dc3dc798e1ef2c1ecd2cbb3b"}],"openstack/image/image_service.py":[{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"5c6ca6c210f4b52777665d639f0402a40eb2f0a4","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    def __init__(self):"},{"line_number":20,"context_line":"        \"\"\"Create an image service.\"\"\""},{"line_number":21,"context_line":"        super(ImageService, self).__init__(service_type\u003d\u0027image\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_960f4b8b","line":21,"updated":"2014-06-23 16:55:15.000000000","message":"Does it make sense to allow specifying this as a class attribute (honestly not sure)","commit_id":"bbe0be6562520cd6dc3dc798e1ef2c1ecd2cbb3b"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"c5c43556e4dacdbd851ad363ac37043f0b4a5138","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    def __init__(self):"},{"line_number":20,"context_line":"        \"\"\"Create an image service.\"\"\""},{"line_number":21,"context_line":"        super(ImageService, self).__init__(service_type\u003d\u0027image\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1ae5cdf2_f397b4e9","line":21,"updated":"2014-06-23 22:39:15.000000000","message":"Alex\u0027s comment on this from the last patch, everything else was addressed:\n\nDoes it make sense to allow specifying this as a class attribute (honestly not sure)","commit_id":"652b7adec75d090ecf70301dd2ef69135dfeecb8"}],"openstack/tests/image/v1/test_image.py":[{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"5c6ca6c210f4b52777665d639f0402a40eb2f0a4","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from openstack.image.v1 import image"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"IDENTIFIER \u003d \u0027IDENTIFIER\u0027"},{"line_number":18,"context_line":"FAKE \u003d {"},{"line_number":19,"context_line":"    \u0027checksum\u0027: \u00271\u0027,"},{"line_number":20,"context_line":"    \u0027container_format\u0027: \u00272\u0027,"},{"line_number":21,"context_line":"    \u0027copy_from\u0027: \u00273\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_7671df0b","line":18,"updated":"2014-06-23 16:55:15.000000000","message":"I think it makes more sense to name this \"example payload\" or something?","commit_id":"bbe0be6562520cd6dc3dc798e1ef2c1ecd2cbb3b"},{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"5c6ca6c210f4b52777665d639f0402a40eb2f0a4","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    \u0027size\u0027: \u002713\u0027,"},{"line_number":33,"context_line":"    \u0027status\u0027: \u002714\u0027,"},{"line_number":34,"context_line":"    \u0027created_at\u0027: \u002715\u0027,"},{"line_number":35,"context_line":"    \u0027updated_at\u0027: \u002716\u0027,"},{"line_number":36,"context_line":"}"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_b662873b","line":35,"updated":"2014-06-23 16:55:15.000000000","message":"These values are properly datetimes right? At some point we might do some validation, it would be good if they were the right format.","commit_id":"bbe0be6562520cd6dc3dc798e1ef2c1ecd2cbb3b"}]}
