)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"9f03fd8be8eca106f10050b963676ade5b1b1ee1","unresolved":false,"context_lines":[{"line_number":11,"context_line":"* Rework tests for requests"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Pinned requests module to \u003c 1.0 as 1.0.2 is now current in pipi"},{"line_number":14,"context_line":"as of 17Dec2012."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Blueprint: tls-verify"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAAM3%2F%2F9iQ%3D","line":14,"updated":"2012-12-18 20:14:39.000000000","message":"Does 1.0 or above cause problems for this patch?  What sort of problems?","commit_id":"20020e6632e2e0401f4691e3e4c0b699aca63453"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"564e009e73e4094c5c13fac0c6dae44111588331","unresolved":false,"context_lines":[{"line_number":11,"context_line":"* Rework tests for requests"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Pinned requests module to \u003c 1.0 as 1.0.2 is now current in pipi"},{"line_number":14,"context_line":"as of 17Dec2012."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Blueprint: tls-verify"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAAM3%2F%2F8Rc%3D","line":14,"in_reply_to":"AAAAM3%2F%2F8aU%3D","updated":"2012-12-19 18:24:01.000000000","message":"OK.  Also, I had a conversation with one of our QE guys yesterday, and I learned that several essential features in prior releases of requests had vanished entirely in the 1.0 releases, so this now seems reasonable to me.","commit_id":"20020e6632e2e0401f4691e3e4c0b699aca63453"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"9ba479fc59d72dbf783ee46404dcde21b833bbf3","unresolved":false,"context_lines":[{"line_number":11,"context_line":"* Rework tests for requests"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Pinned requests module to \u003c 1.0 as 1.0.2 is now current in pipi"},{"line_number":14,"context_line":"as of 17Dec2012."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Blueprint: tls-verify"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAAM3%2F%2F8aU%3D","line":14,"in_reply_to":"AAAAM3%2F%2F9iQ%3D","updated":"2012-12-19 17:17:58.000000000","message":"It was showing some errors about the config argument no longer being valid.  I didn\u0027t want to stop and rework everything for such a new major rewrite of a critical module.  It needs to marinate a bit, there were already two point releases on Monday...","commit_id":"20020e6632e2e0401f4691e3e4c0b699aca63453"}],"novaclient/client.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d29310ff61071666cf456371ee72f53c6297a64b","unresolved":false,"context_lines":[{"line_number":180,"context_line":"            # to check the body.  httplib2 tests check for \u0027Connection refused\u0027"},{"line_number":181,"context_line":"            # or \u0027actively refused\u0027 in the body, so that\u0027s what we\u0027ll do."},{"line_number":182,"context_line":"            if resp.status_code \u003d\u003d 400:"},{"line_number":183,"context_line":"                if \u0027Connection refused\u0027 in resp.text or \\"},{"line_number":184,"context_line":"                   \u0027actively refused\u0027 in resp.text:"},{"line_number":185,"context_line":"                    raise exceptions.ConnectionRefused(resp.text)"},{"line_number":186,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F%2BvU%3D","line":183,"updated":"2012-12-17 23:55:38.000000000","message":"I would prefer that the condition be enclosed in parentheses so that the backslash can be eliminated.","commit_id":"8d2cb38f1c93110c484b63c2941f57bec7b933ef"}],"tests/test_auth_plugins.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d29310ff61071666cf456371ee72f53c6297a64b","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import mock"},{"line_number":17,"context_line":"import pkg_resources"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import requests"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F%2BvA%3D","line":18,"updated":"2012-12-17 23:55:38.000000000","message":"This blank line should be eliminated.","commit_id":"8d2cb38f1c93110c484b63c2941f57bec7b933ef"}],"tests/test_http.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d29310ff61071666cf456371ee72f53c6297a64b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"import mock"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"import requests"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"from novaclient import client"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F%2Bu8%3D","line":2,"updated":"2012-12-17 23:55:38.000000000","message":"This blank line should be eliminated…","commit_id":"8d2cb38f1c93110c484b63c2941f57bec7b933ef"}],"tests/utils.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d29310ff61071666cf456371ee72f53c6297a64b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"import unittest2"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"import requests"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F%2Bus%3D","line":2,"updated":"2012-12-17 23:55:38.000000000","message":"This blank line should be eliminated (and \"import requests\" sorted before \"import unittest2\")","commit_id":"8d2cb38f1c93110c484b63c2941f57bec7b933ef"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"d29310ff61071666cf456371ee72f53c6297a64b","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"class TestResponse(requests.Response):"},{"line_number":14,"context_line":"    \"\"\" Class used to wrap requests.Response and provide some"},{"line_number":15,"context_line":"        convenience to initialize with a dict \"\"\""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"    def __init__(self, data):"},{"line_number":18,"context_line":"        self._text \u003d None"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2F%2Bug%3D","line":15,"updated":"2012-12-17 23:55:38.000000000","message":"1) Eliminate the space after the opening triple-quote;\n2) Don\u0027t indent the second line to line up with the text.\n\nIt would probably be better to format this as:\n\n    \"\"\"\n    Class used to wrap requests.Response and provide some\n    convenience to initialize with a dict.\n    \"\"\"","commit_id":"8d2cb38f1c93110c484b63c2941f57bec7b933ef"}],"tests/v1_1/test_auth.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"0045126c149f44bbbe6067f1a08b2bad7d445e59","unresolved":false,"context_lines":[{"line_number":159,"context_line":"            }"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"            token_url \u003d cs.client.auth_url + \"/tokens\""},{"line_number":162,"context_line":"            kwargs \u003d copy.copy(self.TEST_REQUEST_BASE)"},{"line_number":163,"context_line":"            kwargs[\u0027headers\u0027] \u003d headers"},{"line_number":164,"context_line":"            kwargs[\u0027data\u0027] \u003d json.dumps(body)"},{"line_number":165,"context_line":"            mock_request.assert_called_with("}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2F9%2BM%3D","line":162,"updated":"2012-12-18 16:00:55.000000000","message":"Seems you forgot to import copy.","commit_id":"fd17ab42bb308f8caf190a83fd82ec37b9956ebe"}]}