)]}'
{"nova/tests/unit/virt/vmwareapi/test_read_write_util.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"7698704b789972d867781c913d39a0d211d06a2c","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from nova import test"},{"line_number":21,"context_line":"from nova.virt.vmwareapi import read_write_util"},{"line_number":22,"context_line":"import urllib"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"CONF \u003d cfg.CONF"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9a80dd14_1d42233e","line":22,"updated":"2015-03-12 19:09:21.000000000","message":"this is not a nova import. it should be in the above section","commit_id":"5cb3abb2bc4217a01150c2034234027be18c45b5"},{"author":{"_account_id":14819,"name":"Timofey Durakov","email":"timofei.nd@gmail.com","username":"tdurakov"},"change_message_id":"a095ee3c5481b77d1f86f071d99172487c70637a","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from nova import test"},{"line_number":21,"context_line":"from nova.virt.vmwareapi import read_write_util"},{"line_number":22,"context_line":"import urllib"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"CONF \u003d cfg.CONF"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9a80dd14_48f6e091","line":22,"in_reply_to":"9a80dd14_1d42233e","updated":"2015-03-17 17:56:36.000000000","message":"garyk: Thanks for comment. Checked pep8, everything looks fine. Could you explain what\u0027s wrong with this import?","commit_id":"5cb3abb2bc4217a01150c2034234027be18c45b5"},{"author":{"_account_id":8412,"name":"Mike Durnosvistov","email":"glacierr.dev@gmail.com","username":"mdurnosvistov"},"change_message_id":"62f46d5aa5326395f084b8161f234bc9de1d9166","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from nova import test"},{"line_number":21,"context_line":"from nova.virt.vmwareapi import read_write_util"},{"line_number":22,"context_line":"import urllib"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"CONF \u003d cfg.CONF"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"da9b358b_53f85149","line":22,"updated":"2015-03-31 14:30:46.000000000","message":"Imports should be grouped in the following order:\n\n 1) standard library imports\n 2) related third party imports\n 3) local application/library specific imports\n\nhttps://www.python.org/dev/peps/pep-0008/#imports","commit_id":"55b5a8a141ab4557ca5f74678da2626d60f54f89"},{"author":{"_account_id":14819,"name":"Timofey Durakov","email":"timofei.nd@gmail.com","username":"tdurakov"},"change_message_id":"b47e885216cb9490cf8090610185aaababd460e6","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from nova import test"},{"line_number":21,"context_line":"from nova.virt.vmwareapi import read_write_util"},{"line_number":22,"context_line":"import urllib"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"CONF \u003d cfg.CONF"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"da9b358b_223f95d7","line":22,"in_reply_to":"da9b358b_53f85149","updated":"2015-04-01 08:27:31.000000000","message":"Thanks for comment, Mike. Relied on PyCharm too much. No alerts, pep8 check keep silence as well. Fixed.","commit_id":"55b5a8a141ab4557ca5f74678da2626d60f54f89"}]}