)]}'
{"novaclient/client.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"698258319081fb93eaa168b007bbe056e9fe5e79","unresolved":false,"context_lines":[{"line_number":21,"context_line":"\"\"\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"import logging"},{"line_number":24,"context_line":"import requests"},{"line_number":25,"context_line":"import time"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9Upk%3D","line":24,"updated":"2013-12-12 17:13:21.000000000","message":"requests is a third-party import and should not have been sorted into the first group of imports.","commit_id":"fe30feaf04b7871dcaedbec5ff8a9cc5f51dbb69"}],"novaclient/tests/test_discover.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"698258319081fb93eaa168b007bbe056e9fe5e79","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import imp"},{"line_number":17,"context_line":"import inspect"},{"line_number":18,"context_line":"import mock"},{"line_number":19,"context_line":"import pkg_resources"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"import novaclient.shell"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9UpU%3D","line":18,"updated":"2013-12-12 17:13:21.000000000","message":"mock and pkg_resources are third-party imports, and so should be separated from the first group of standard library imports by a blank line.","commit_id":"fe30feaf04b7871dcaedbec5ff8a9cc5f51dbb69"}],"novaclient/tests/v1_1/test_auth.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"698258319081fb93eaa168b007bbe056e9fe5e79","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"import copy"},{"line_number":15,"context_line":"import json"},{"line_number":16,"context_line":"import mock"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import requests"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9UpM%3D","line":16,"updated":"2013-12-12 17:13:21.000000000","message":"Not your fault, but mock should be in the group with \"import requests\", not with the standard library imports.","commit_id":"fe30feaf04b7871dcaedbec5ff8a9cc5f51dbb69"}],"novaclient/tests/v1_1/test_shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"698258319081fb93eaa168b007bbe056e9fe5e79","unresolved":false,"context_lines":[{"line_number":19,"context_line":"import base64"},{"line_number":20,"context_line":"import datetime"},{"line_number":21,"context_line":"import fixtures"},{"line_number":22,"context_line":"import mock"},{"line_number":23,"context_line":"import novaclient.client"},{"line_number":24,"context_line":"from novaclient import exceptions"},{"line_number":25,"context_line":"from novaclient.openstack.common import timeutils"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9Uow%3D","line":22,"updated":"2013-12-12 17:13:21.000000000","message":"fixtures and mock are third-party imports, and should not be included in the first group of imports.","commit_id":"fe30feaf04b7871dcaedbec5ff8a9cc5f51dbb69"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"698258319081fb93eaa168b007bbe056e9fe5e79","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from novaclient.tests.v1_1 import fakes"},{"line_number":29,"context_line":"import novaclient.v1_1.shell"},{"line_number":30,"context_line":"import os"},{"line_number":31,"context_line":"import six"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"class ShellFixture(fixtures.Fixture):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9Uoo%3D","line":31,"updated":"2013-12-12 17:13:21.000000000","message":"os is a standard library import, and so should be in the first group of imports.  six is a third-party import, and so should be in the second group of imports with the imports for mock and fixtures.","commit_id":"fe30feaf04b7871dcaedbec5ff8a9cc5f51dbb69"}]}