)]}'
{"tempest/services/nova/json/security_groups_client.py":[{"author":{"_account_id":97,"name":"Daryl Walleck","email":"daryl.walleck@rackspace.com","username":"dwalleck"},"change_message_id":"3d0a9221e8eaa25be245b10a0fddd1ebda61f749","unresolved":false,"context_lines":[{"line_number":18,"context_line":"            for param, value in params.iteritems():"},{"line_number":19,"context_line":"                param_list.append(\"%s\u003d%s\u0026\" % (param, value))"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"            url \u003d \"os-security-groups?\" + \"\".join(param_list)"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"        resp, body \u003d self.client.get(url)"},{"line_number":24,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAD3%2F%2F9y8%3D","line":21,"updated":"2012-01-16 19:22:40.000000000","message":"He\u0027s following the style from the other clients, so blame that on me :) Still, your suggestion makes sense","commit_id":"9ce6fd3056746816feecbd61887bcdfb26347cd0"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"170a57f9cbc20155fd71ea7fc6f86cac63047c47","unresolved":false,"context_lines":[{"line_number":18,"context_line":"            for param, value in params.iteritems():"},{"line_number":19,"context_line":"                param_list.append(\"%s\u003d%s\u0026\" % (param, value))"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"            url \u003d \"os-security-groups?\" + \"\".join(param_list)"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"        resp, body \u003d self.client.get(url)"},{"line_number":24,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAD3%2F%2F%2BJI%3D","line":21,"updated":"2012-01-13 23:01:19.000000000","message":"might be cleaner to do:\n\n url +\u003d \u0027?\u0027 + \u0027 \u0027.join(param_list)","commit_id":"9ce6fd3056746816feecbd61887bcdfb26347cd0"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"170a57f9cbc20155fd71ea7fc6f86cac63047c47","unresolved":false,"context_lines":[{"line_number":33,"context_line":"            for param, value in params.iteritems():"},{"line_number":34,"context_line":"                param_list.append(\"%s\u003d%s\u0026\" % (param, value))"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"            url \u003d \"os-security-groups/detail?\" + \"\".join(param_list)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        resp, body \u003d self.client.get(url)"},{"line_number":39,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAD3%2F%2F%2BJM%3D","line":36,"updated":"2012-01-13 23:01:19.000000000","message":"might be cleaner to do:\n\n url +\u003d \u0027?\u0027 + \u0027 \u0027.join(param_list)","commit_id":"9ce6fd3056746816feecbd61887bcdfb26347cd0"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"0a2e4be5fda8644a66b8b9eb19ba3f9771758daf","unresolved":false,"context_lines":[{"line_number":18,"context_line":"            for param, value in params.iteritems():"},{"line_number":19,"context_line":"                param_list.append(\"%s\u003d%s\u0026\" % (param, value))"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"            url +\u003d \u0027?\u0027 +\u0027 \u0027.join(param_list)"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"        resp, body \u003d self.client.get(url)"},{"line_number":24,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAD3%2F%2F9SI%3D","line":21,"updated":"2012-01-17 18:17:37.000000000","message":"pep8 here.. need a space between + and \u0027 \u0027","commit_id":"f0e653d0499bfd1bab6a11a2b05ebc1f6b34465a"}]}