)]}'
{"ironicclient/common/http.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"3105be75ac6f7b2b65602551900778e6c6396574","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import copy"},{"line_number":17,"context_line":"import logging"},{"line_number":18,"context_line":"import os"},{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":"import socket"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from ironicclient.openstack.common.py3kcompat import urlutils"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FTgE%3D","line":19,"updated":"2013-11-11 11:01:06.000000000","message":"Maybe also fixing this import here? Since six is not part of the standard libs it should be in another group, as per hacking and PEP8. (Also will need to update the commit message)\n\nhttp://www.python.org/dev/peps/pep-0008/#imports","commit_id":"8fefacfbaec08117baf43c42001350b987dc24ee"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"6c82351fe2ed2cb46d8427484f8ab239cf5378ba","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import copy"},{"line_number":17,"context_line":"import logging"},{"line_number":18,"context_line":"import os"},{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":"import socket"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from ironicclient.openstack.common.py3kcompat import urlutils"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FTdc%3D","line":19,"in_reply_to":"AAAATn%2F%2FTfU%3D","updated":"2013-11-11 11:21:42.000000000","message":"That works as well. But I think the patch should be more generic not only fixing the six but other third-part libs as well","commit_id":"8fefacfbaec08117baf43c42001350b987dc24ee"},{"author":{"_account_id":1726,"name":"Ghe Rivero","email":"ghe.rivero@gmail.com","username":"ghe.rivero"},"change_message_id":"cec1dbfb1a050604798dbb9cbdfe4c97db17cbed","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import copy"},{"line_number":17,"context_line":"import logging"},{"line_number":18,"context_line":"import os"},{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":"import socket"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from ironicclient.openstack.common.py3kcompat import urlutils"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FTfU%3D","line":19,"in_reply_to":"AAAATn%2F%2FTgE%3D","updated":"2013-11-11 11:06:13.000000000","message":"Uhm... there are more of these, so what about preparing a patch just for that? .","commit_id":"8fefacfbaec08117baf43c42001350b987dc24ee"}]}