)]}'
{"tests/v1_1/test_hosts.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"3b17c8937deddb2367144588312e9151a649eed2","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        cs.assert_called("},{"line_number":54,"context_line":"                            \u0027POST\u0027,"},{"line_number":55,"context_line":"                            \u0027/os-hosts/sample_host/action\u0027,"},{"line_number":56,"context_line":"                            {\u0027startup\u0027: None})"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def test_host_reboot(self):"},{"line_number":59,"context_line":"        host \u003d cs.hosts.get(\u0027sample_host\u0027)[0]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FOOY%3D","line":56,"updated":"2013-02-01 15:19:09.000000000","message":"This isn\u0027t what I would want to see.  Reasonable alternatives are:\n\n1. Move the whole \"\u0027POST\u0027, …)\" onto the next line, indented by four spaces more than the existing indentation.\n\n    cs.assert_called(\n        \u0027POST\u0027, \u0027/os-hosts/sample_host/action\u0027, {\u0027startup\u0027: None})\n\n2. Move only the \"{\u0027startup\u0027: None})\" onto the next line, indented to begin at the column just after the opening parenthesis.\n\n    cs.assert_called(\u0027POST\u0027, \u0027/os-hosts/sample_host/action\u0027,\n                     {\u0027startup\u0027: None})","commit_id":"098e422f3f242e7e95a63d07607fa1261851bffa"}],"tests/v1_1/test_shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"3b17c8937deddb2367144588312e9151a649eed2","unresolved":false,"context_lines":[{"line_number":509,"context_line":""},{"line_number":510,"context_line":"    def test_usage(self):"},{"line_number":511,"context_line":"        self.run_command(\u0027usage --start 2000-01-20 --end 2005-02-01 \\"},{"line_number":512,"context_line":"                             --tenant test\u0027)"},{"line_number":513,"context_line":"        self.assert_called(\u0027GET\u0027,"},{"line_number":514,"context_line":"                           \u0027/os-simple-tenant-usage/test?\u0027 +"},{"line_number":515,"context_line":"                           \u0027start\u003d2000-01-20T00:00:00\u0026\u0027 +"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FOOU%3D","line":512,"updated":"2013-02-01 15:19:09.000000000","message":"There is no need for this backslash.  Use adjacent string concatenation, like so:\n\n    self.run_command(\u0027usage --start 2000-01-20 --end 2005-02-01 \u0027\n                     \u0027--tenant test\u0027)","commit_id":"098e422f3f242e7e95a63d07607fa1261851bffa"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"3b17c8937deddb2367144588312e9151a649eed2","unresolved":false,"context_lines":[{"line_number":674,"context_line":"        self.assert_called("},{"line_number":675,"context_line":"                            \u0027POST\u0027,"},{"line_number":676,"context_line":"                            \u0027/os-hosts/sample-host/action\u0027,"},{"line_number":677,"context_line":"                            {\u0027startup\u0027: None})"},{"line_number":678,"context_line":""},{"line_number":679,"context_line":"    def test_host_shutdown(self):"},{"line_number":680,"context_line":"        self.run_command(\u0027host-action sample-host --action shutdown\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FOOQ%3D","line":677,"updated":"2013-02-01 15:19:09.000000000","message":"See my comment on test_hosts.py.","commit_id":"098e422f3f242e7e95a63d07607fa1261851bffa"}]}