)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"852377feec251eb3b6cab9e52d980ed1bd6d8172","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Get rid of contextlib.nested() for py3"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"contextlib.nested() is deprecated in Python 3, and it spews copious"},{"line_number":10,"context_line":"warnings to make sure you know this."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Since 2.7, we can use multiple context managers in a \u0027with\u0027 statement,"},{"line_number":13,"context_line":"like so:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a29b11f_b4b14a49","line":10,"updated":"2015-10-21 20:37:30.000000000","message":"It\u0027s not only deprecated, it was also removed in Python 3.0.","commit_id":"9e493c7bf7ddb11937f1b468e45109ea31fcccbd"},{"author":{"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},"change_message_id":"5f2a43e1ae907879fddbb8ccb53f0c3e3cbc709c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Get rid of contextlib.nested() for py3"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"contextlib.nested() is deprecated in Python 3, and it spews copious"},{"line_number":10,"context_line":"warnings to make sure you know this."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Since 2.7, we can use multiple context managers in a \u0027with\u0027 statement,"},{"line_number":13,"context_line":"like so:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a29b11f_8fa8bb78","line":10,"in_reply_to":"3a29b11f_b4b14a49","updated":"2015-10-21 21:00:36.000000000","message":"Done","commit_id":"9e493c7bf7ddb11937f1b468e45109ea31fcccbd"}],"test/unit/account/test_reaper.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"852377feec251eb3b6cab9e52d980ed1bd6d8172","unresolved":false,"context_lines":[{"line_number":416,"context_line":"        self.max_delete_fail \u003d 0"},{"line_number":417,"context_line":"        with patch(\u0027swift.account.reaper.direct_get_container\u0027,"},{"line_number":418,"context_line":"                   self.fake_direct_get_container), \\"},{"line_number":419,"context_line":"                patch(\u0027swift.account.reaper.direct_delete_container\u0027,"},{"line_number":420,"context_line":"                      self.fake_direct_delete_container), \\"},{"line_number":421,"context_line":"                patch(\u0027swift.account.reaper.AccountReaper.get_container_ring\u0027,"},{"line_number":422,"context_line":"                      self.fake_container_ring), \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a29b11f_74ef32e9","line":419,"updated":"2015-10-21 20:37:30.000000000","message":"Hum, the indentation is surprising. I would expect calls to patch() to be aligned vertically.","commit_id":"9e493c7bf7ddb11937f1b468e45109ea31fcccbd"},{"author":{"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},"change_message_id":"5f2a43e1ae907879fddbb8ccb53f0c3e3cbc709c","unresolved":false,"context_lines":[{"line_number":416,"context_line":"        self.max_delete_fail \u003d 0"},{"line_number":417,"context_line":"        with patch(\u0027swift.account.reaper.direct_get_container\u0027,"},{"line_number":418,"context_line":"                   self.fake_direct_get_container), \\"},{"line_number":419,"context_line":"                patch(\u0027swift.account.reaper.direct_delete_container\u0027,"},{"line_number":420,"context_line":"                      self.fake_direct_delete_container), \\"},{"line_number":421,"context_line":"                patch(\u0027swift.account.reaper.AccountReaper.get_container_ring\u0027,"},{"line_number":422,"context_line":"                      self.fake_container_ring), \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a29b11f_cfa23399","line":419,"in_reply_to":"3a29b11f_74ef32e9","updated":"2015-10-21 21:00:36.000000000","message":"Yep, me too. However, if the indentation is not a multiple of 4, pep8 or flake8 or some other linter we use won\u0027t let it pass. I had a patch up once to disable that particular check, but nobody else liked it.","commit_id":"9e493c7bf7ddb11937f1b468e45109ea31fcccbd"}]}