)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"change_message_id":"4bebaa069a1c974b772ac570837d300869042dec","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     LiuYan \u003clykobe@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-07-23 05:12:03 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"remove redundant commas"},{"line_number":8,"context_line":"Fixes Bug1477142"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I20da35b15f6ac734f725059562189c8468fc4cd6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_854184ab","line":7,"updated":"2015-07-22 13:59:54.000000000","message":"You need a blank line between the commit title and the body.","commit_id":"cf24254820eef16e5a3e22681d0435c769c223a5"},{"author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"change_message_id":"4bebaa069a1c974b772ac570837d300869042dec","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-07-23 05:12:03 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"remove redundant commas"},{"line_number":8,"context_line":"Fixes Bug1477142"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I20da35b15f6ac734f725059562189c8468fc4cd6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_651b18cb","line":8,"updated":"2015-07-22 13:59:54.000000000","message":"The usual syntax for this is\n\n    Closes-Bug: #1477142\n\nYou should also add some information about the rationale behind the change.","commit_id":"cf24254820eef16e5a3e22681d0435c769c223a5"},{"author":{"_account_id":7575,"name":"Sabari","email":"smurugesan@vmware.com","username":"sabari"},"change_message_id":"5519d056694d139f2193a7d3a296424eb974dbb6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"remove redundant commas and round brackets"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"File glance/registry/api/v1/images.py:"},{"line_number":10,"context_line":"Line 289: msg \u003d _(\"Unsupported sort_key. Acceptable values: %s\") % (_keys,)"},{"line_number":11,"context_line":"Line 298: msg \u003d _(\"Unsupported sort_dir. Acceptable values: %s\") % (_keys,)"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"redundant comma and round brackets should be remove, i.e."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3a50d1a3_3600be65","line":10,"updated":"2015-07-23 05:59:34.000000000","message":"You don\u0027t have to include your actual code changes in the commit message. \n\nI think what Louis meant was something descriptive like \"_keys is always a string and hence there is no need to create a single-item tuple for string formatting\" or something better :)","commit_id":"2786614302254e8880d6aad066d97ed1fc9abc3a"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"54e8268339eeaf73695fdf823d90447f5a2f0292","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"remove redundant commas and round brackets"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"_keys is always a string and hence there is no need to create a "},{"line_number":10,"context_line":"single-item tuple for string formatting."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug: 1477142"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3a50d1a3_e2de05dc","line":9,"updated":"2015-07-23 10:51:23.000000000","message":"remove blank space at the end of line","commit_id":"ed4a6f6cad6a71f1a2dfe33af9d9a47350819640"}],"glance/registry/api/v1/images.py":[{"author":{"_account_id":11391,"name":"Mikhail Fedosin","email":"mfedosin@redhat.com","username":"fedosinme"},"change_message_id":"355bc86b816057c31119535596a7355cf770f75f","unresolved":false,"context_lines":[{"line_number":286,"context_line":"        sort_key \u003d req.params.get(\u0027sort_key\u0027, \u0027created_at\u0027)"},{"line_number":287,"context_line":"        if sort_key is not None and sort_key not in SUPPORTED_SORT_KEYS:"},{"line_number":288,"context_line":"            _keys \u003d \u0027, \u0027.join(SUPPORTED_SORT_KEYS)"},{"line_number":289,"context_line":"            msg \u003d _(\"Unsupported sort_key. Acceptable values: %s\") % (_keys)"},{"line_number":290,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":291,"context_line":"        return sort_key"},{"line_number":292,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_edd94e58","line":289,"updated":"2015-07-22 14:28:10.000000000","message":"If you do that then remove round brackets too, i.e.\n\nmsg \u003d _(\"Unsupported sort_key. Acceptable values: %s\") % _keys","commit_id":"cf24254820eef16e5a3e22681d0435c769c223a5"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"54e8268339eeaf73695fdf823d90447f5a2f0292","unresolved":false,"context_lines":[{"line_number":286,"context_line":"        sort_key \u003d req.params.get(\u0027sort_key\u0027, \u0027created_at\u0027)"},{"line_number":287,"context_line":"        if sort_key is not None and sort_key not in SUPPORTED_SORT_KEYS:"},{"line_number":288,"context_line":"            _keys \u003d \u0027, \u0027.join(SUPPORTED_SORT_KEYS)"},{"line_number":289,"context_line":"            msg \u003d _(\"Unsupported sort_key. Acceptable values: %s\") % _keys"},{"line_number":290,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":291,"context_line":"        return sort_key"},{"line_number":292,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"3a50d1a3_08f553eb","line":289,"updated":"2015-07-23 10:51:23.000000000","message":"there are lot many occurrences of same,\n\ne.g. https://github.com/openstack/glance/blob/master/glance/api/v1/images.py#L629","commit_id":"ed4a6f6cad6a71f1a2dfe33af9d9a47350819640"}]}