)]}'
{"openstack_dashboard/dashboards/admin/networks/ports/tables.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"dc0d9eecbd6758e0e824ef8278d87d26cf9c3c7e","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        try:"},{"line_number":39,"context_line":"            api.neutron.port_delete(request, obj_id)"},{"line_number":40,"context_line":"        except Exception as e:"},{"line_number":41,"context_line":"            msg \u003d _(\u0027Failed to delete port %s\u0027) % e"},{"line_number":42,"context_line":"            LOG.info(msg)"},{"line_number":43,"context_line":"            network_id \u003d self.table.kwargs[\u0027network_id\u0027]"},{"line_number":44,"context_line":"            redirect \u003d reverse(\u0027horizon:admin:networks:detail\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BI0A%3D","line":41,"updated":"2013-12-02 05:04:41.000000000","message":"Please add a colon or a period before \"%s\" because %s is replaced with a reason of the exception.","commit_id":"b702b61d71be0aaa89e4c751c4fa27635bfeee82"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"8b693e98d71b4beea73ea8c6b4df029eefe35e56","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        try:"},{"line_number":39,"context_line":"            api.neutron.port_delete(request, obj_id)"},{"line_number":40,"context_line":"        except Exception as e:"},{"line_number":41,"context_line":"            msg \u003d _(\u0027Failed to delete port : \"%s\"\u0027) % e"},{"line_number":42,"context_line":"            LOG.info(msg)"},{"line_number":43,"context_line":"            network_id \u003d self.table.kwargs[\u0027network_id\u0027]"},{"line_number":44,"context_line":"            redirect \u003d reverse(\u0027horizon:admin:networks:detail\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F97v0%3D","line":41,"updated":"2013-12-04 09:51:25.000000000","message":"Akihiro, did you really mean this? Adding a colon (and a whitespace between text and colon) breaks with the rest of out exception messages.","commit_id":"1a3be1c1a15d493e72c253ef1f76027ac3d14d24"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c3b14fa4b944508e26e80c316536bf8e47272763","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        try:"},{"line_number":39,"context_line":"            api.neutron.port_delete(request, obj_id)"},{"line_number":40,"context_line":"        except Exception as e:"},{"line_number":41,"context_line":"            msg \u003d _(\u0027Failed to delete port : \"%s\"\u0027) % e"},{"line_number":42,"context_line":"            LOG.info(msg)"},{"line_number":43,"context_line":"            network_id \u003d self.table.kwargs[\u0027network_id\u0027]"},{"line_number":44,"context_line":"            redirect \u003d reverse(\u0027horizon:admin:networks:detail\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F90Dw%3D","line":41,"in_reply_to":"AAAATn%2F941A%3D","updated":"2013-12-05 16:42:29.000000000","message":"Agree with Lin. I just suggest to adding a colon. Our convention is no space before a colon and one space after a colon and we should follow it. double quotes around error message are unnecessary.","commit_id":"1a3be1c1a15d493e72c253ef1f76027ac3d14d24"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"64004e8241ec407c2152ca27ec17838b8001b0b8","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        try:"},{"line_number":39,"context_line":"            api.neutron.port_delete(request, obj_id)"},{"line_number":40,"context_line":"        except Exception as e:"},{"line_number":41,"context_line":"            msg \u003d _(\u0027Failed to delete port : \"%s\"\u0027) % e"},{"line_number":42,"context_line":"            LOG.info(msg)"},{"line_number":43,"context_line":"            network_id \u003d self.table.kwargs[\u0027network_id\u0027]"},{"line_number":44,"context_line":"            redirect \u003d reverse(\u0027horizon:admin:networks:detail\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F941A%3D","line":41,"in_reply_to":"AAAATn%2F97v0%3D","updated":"2013-12-05 10:00:38.000000000","message":"Should this just be \"..port: %s\" - no whitespace between text and colon, and no double quote on error message.","commit_id":"1a3be1c1a15d493e72c253ef1f76027ac3d14d24"}],"requirements.txt":[{"author":{"_account_id":6914,"name":"Tatiana Ovchinnikova","email":"t.v.ovtchinnikova@gmail.com","username":"tmazur"},"change_message_id":"f526a90d5e26df9965bb4f50329cf510f103c011","unresolved":false,"context_lines":[{"line_number":16,"context_line":"python-neutronclient\u003e\u003d2.3.0,\u003c3"},{"line_number":17,"context_line":"python-swiftclient\u003e\u003d1.5"},{"line_number":18,"context_line":"python-ceilometerclient\u003e\u003d1.0.6"},{"line_number":19,"context_line":"python-troveclient\u003c1"},{"line_number":20,"context_line":"pytz\u003e\u003d2010h"},{"line_number":21,"context_line":"# Horizon Utility Requirements"},{"line_number":22,"context_line":"# for SECURE_KEY generation"}],"source_content_type":"text/plain","patch_set":2,"id":"AAAATn%2F9%2B%2F8%3D","line":19,"updated":"2013-12-03 19:04:34.000000000","message":"Why did you change this?","commit_id":"705340991a0638c0a97227b6339a8841340713f0"}]}