)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"b1ec580a88b3d15550374ace3997930309b7f5c1","unresolved":false,"context_lines":[{"line_number":7,"context_line":"python3: Fix unicode compatibility python2/python3"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Python3 enforces the distinction between byte strings and text strings"},{"line_number":10,"context_line":"more rigorously than python2. So use six.text_type, six.u()"},{"line_number":11,"context_line":"where approiate."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I890e19cb857e10f0292aabdaebaa8e7a7bd8db23"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAPn%2F%2Ft40%3D","line":10,"updated":"2013-06-24 15:45:15.000000000","message":"s/, / and / (femtonit)","commit_id":"40f99c4415fe087f181fec5d1477b8daf3546108"},{"author":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"change_message_id":"81112214a3d673f3976e9e33f00371b54eeca0e0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"python3: Fix unicode compatibility python2/python3"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Python3 enforces the distinction between byte strings and text strings"},{"line_number":10,"context_line":"more rigorously than python2. So use six.text_type, six.u()"},{"line_number":11,"context_line":"where approiate."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I890e19cb857e10f0292aabdaebaa8e7a7bd8db23"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAPn%2F%2FtuA%3D","line":10,"in_reply_to":"AAAAPn%2F%2Ft40%3D","updated":"2013-06-24 16:47:52.000000000","message":"Done","commit_id":"40f99c4415fe087f181fec5d1477b8daf3546108"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"b1ec580a88b3d15550374ace3997930309b7f5c1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Python3 enforces the distinction between byte strings and text strings"},{"line_number":10,"context_line":"more rigorously than python2. So use six.text_type, six.u()"},{"line_number":11,"context_line":"where approiate."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I890e19cb857e10f0292aabdaebaa8e7a7bd8db23"},{"line_number":14,"context_line":"Signed-off-by: Chuck Short \u003cchuck.short@canonical.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAPn%2F%2Ft4w%3D","line":11,"updated":"2013-06-24 15:45:15.000000000","message":"s/approiate/appropriate/ (femtonit)","commit_id":"40f99c4415fe087f181fec5d1477b8daf3546108"},{"author":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"change_message_id":"81112214a3d673f3976e9e33f00371b54eeca0e0","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Python3 enforces the distinction between byte strings and text strings"},{"line_number":10,"context_line":"more rigorously than python2. So use six.text_type, six.u()"},{"line_number":11,"context_line":"where approiate."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I890e19cb857e10f0292aabdaebaa8e7a7bd8db23"},{"line_number":14,"context_line":"Signed-off-by: Chuck Short \u003cchuck.short@canonical.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAPn%2F%2FtuE%3D","line":11,"in_reply_to":"AAAAPn%2F%2Ft4w%3D","updated":"2013-06-24 16:47:52.000000000","message":"Done","commit_id":"40f99c4415fe087f181fec5d1477b8daf3546108"}],"novaclient/shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"b1ec580a88b3d15550374ace3997930309b7f5c1","unresolved":false,"context_lines":[{"line_number":28,"context_line":"import os"},{"line_number":29,"context_line":"import pkg_resources"},{"line_number":30,"context_line":"import pkgutil"},{"line_number":31,"context_line":"import six"},{"line_number":32,"context_line":"import sys"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"HAS_KEYRING \u003d False"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2Ft4o%3D","line":31,"updated":"2013-06-24 15:45:15.000000000","message":"\"six\" is a third-party import; please put it in an import group of its own…and if you could move the pkg_resources import when you do so, that would be helpful.","commit_id":"40f99c4415fe087f181fec5d1477b8daf3546108"},{"author":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"change_message_id":"81112214a3d673f3976e9e33f00371b54eeca0e0","unresolved":false,"context_lines":[{"line_number":28,"context_line":"import os"},{"line_number":29,"context_line":"import pkg_resources"},{"line_number":30,"context_line":"import pkgutil"},{"line_number":31,"context_line":"import six"},{"line_number":32,"context_line":"import sys"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"HAS_KEYRING \u003d False"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2Ftt4%3D","line":31,"in_reply_to":"AAAAPn%2F%2Ft4o%3D","updated":"2013-06-24 16:47:52.000000000","message":"Done","commit_id":"40f99c4415fe087f181fec5d1477b8daf3546108"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"b1ec580a88b3d15550374ace3997930309b7f5c1","unresolved":false,"context_lines":[{"line_number":767,"context_line":""},{"line_number":768,"context_line":"    except Exception as e:"},{"line_number":769,"context_line":"        logger.debug(e, exc_info\u003d1)"},{"line_number":770,"context_line":"        print(\"ERROR: %s\" % strutils.safe_encode(six.text_type(e)),"},{"line_number":771,"context_line":"              file\u003dsys.stderr)"},{"line_number":772,"context_line":"        sys.exit(1)"},{"line_number":773,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2Ft4I%3D","line":770,"updated":"2013-06-24 15:45:15.000000000","message":"You know that strutils.safe_encode() encodes unicode into utf-8, right?  The original code appears to be trying to transcode from str to unicode back to str, so I\u0027m not 100% certain I understand what it\u0027s doing; I just want to make sure you\u0027ve thought through any subtleties :)","commit_id":"40f99c4415fe087f181fec5d1477b8daf3546108"},{"author":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"change_message_id":"81112214a3d673f3976e9e33f00371b54eeca0e0","unresolved":false,"context_lines":[{"line_number":767,"context_line":""},{"line_number":768,"context_line":"    except Exception as e:"},{"line_number":769,"context_line":"        logger.debug(e, exc_info\u003d1)"},{"line_number":770,"context_line":"        print(\"ERROR: %s\" % strutils.safe_encode(six.text_type(e)),"},{"line_number":771,"context_line":"              file\u003dsys.stderr)"},{"line_number":772,"context_line":"        sys.exit(1)"},{"line_number":773,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2FtvE%3D","line":770,"in_reply_to":"AAAAPn%2F%2Ft4I%3D","updated":"2013-06-24 16:47:52.000000000","message":"Yeah Ill be fixing this in a later in another set.","commit_id":"40f99c4415fe087f181fec5d1477b8daf3546108"}],"requirements.txt":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"b1ec580a88b3d15550374ace3997930309b7f5c1","unresolved":false,"context_lines":[{"line_number":5,"context_line":"prettytable\u003e\u003d0.6,\u003c0.8"},{"line_number":6,"context_line":"requests\u003e\u003d0.8"},{"line_number":7,"context_line":"simplejson"},{"line_number":8,"context_line":"six"}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAPn%2F%2Ft30%3D","line":8,"updated":"2013-06-24 15:45:15.000000000","message":"I\u0027m guessing you\u0027re prepared to rebase your patches depending on which one gets in first?","commit_id":"40f99c4415fe087f181fec5d1477b8daf3546108"},{"author":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"change_message_id":"81112214a3d673f3976e9e33f00371b54eeca0e0","unresolved":false,"context_lines":[{"line_number":5,"context_line":"prettytable\u003e\u003d0.6,\u003c0.8"},{"line_number":6,"context_line":"requests\u003e\u003d0.8"},{"line_number":7,"context_line":"simplejson"},{"line_number":8,"context_line":"six"}],"source_content_type":"text/plain","patch_set":1,"id":"AAAAPn%2F%2FtvQ%3D","line":8,"in_reply_to":"AAAAPn%2F%2Ft30%3D","updated":"2013-06-24 16:47:52.000000000","message":"Yes","commit_id":"40f99c4415fe087f181fec5d1477b8daf3546108"}]}