)]}'
{"novaclient/tests/unit/v2/test_shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"fb468793f9ac03cb504b49fd53d5a8e5b5fe64ce","unresolved":false,"context_lines":[{"line_number":2435,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/os-server-groups\u0027,"},{"line_number":2436,"context_line":"                           {\u0027server_group\u0027: {\u0027name\u0027: \u0027wjsg\u0027,"},{"line_number":2437,"context_line":"                                             \u0027policies\u0027: [\u0027affinity\u0027]}},"},{"line_number":2438,"context_line":"                                                                pos\u003d0)"},{"line_number":2439,"context_line":""},{"line_number":2440,"context_line":"    def test_delete_multi_server_groups(self):"},{"line_number":2441,"context_line":"        self.run_command(\u0027server-group-delete 12345 56789\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a4dcd0f_ee743d9d","line":2438,"updated":"2015-08-04 18:42:14.000000000","message":"This is the source of your pep8 error.  \u0027pos\u0027 is an argument to the assert_called(), and so should be indented so that it lines up immediately after the \u0027(\u0027 on line 2435.","commit_id":"6c9f7045912b6fea5548dc55522b32c68fb5927a"}],"novaclient/v2/shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"fb468793f9ac03cb504b49fd53d5a8e5b5fe64ce","unresolved":false,"context_lines":[{"line_number":4381,"context_line":""},{"line_number":4382,"context_line":""},{"line_number":4383,"context_line":"def _print_server_group_details(server_group):"},{"line_number":4384,"context_line":"    #This is for compatible with Nova v2 API, remove after v2"},{"line_number":4385,"context_line":"    #is dropped."},{"line_number":4386,"context_line":"    try:"},{"line_number":4387,"context_line":"        server_group[0].project_id"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a4dcd0f_ee9bddc7","line":4384,"updated":"2015-08-04 18:42:14.000000000","message":"Please put a space after the \u0027#\u0027 in this comment block. (femtonit, since pep8 didn\u0027t appear to complain about it, but it is a pep8 rule.)","commit_id":"6c9f7045912b6fea5548dc55522b32c68fb5927a"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"fb468793f9ac03cb504b49fd53d5a8e5b5fe64ce","unresolved":false,"context_lines":[{"line_number":4387,"context_line":"        server_group[0].project_id"},{"line_number":4388,"context_line":"        columns \u003d [\u0027Id\u0027, \u0027Name\u0027, \u0027Project_id\u0027, \u0027Policies\u0027,"},{"line_number":4389,"context_line":"                   \u0027Members\u0027, \u0027Metadata\u0027]"},{"line_number":4390,"context_line":"    except:"},{"line_number":4391,"context_line":"        columns \u003d [\u0027Id\u0027, \u0027Name\u0027, \u0027Policies\u0027, \u0027Members\u0027, \u0027Metadata\u0027]"},{"line_number":4392,"context_line":"    utils.print_list(server_group, columns)"},{"line_number":4393,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a4dcd0f_aef39509","line":4390,"updated":"2015-08-04 18:42:14.000000000","message":"No bare \u0027except\u0027 clauses; at a minimum, we would want to see \"except Exception\".  However, in this case, don\u0027t use a try/except at all; use hasattr() to test for the existence of the \u0027project_id\u0027 attribute.","commit_id":"6c9f7045912b6fea5548dc55522b32c68fb5927a"}]}