)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"351289953fec19e0993aace2b0f39abe57308fc6","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2013-12-26 01:20:29 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix wrong message of MACAlreadyExists"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #1264127"},{"line_number":10,"context_line":"Change-Id: I315aa16fd82485b13d8d3bda56a2e0d1f368b937"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAT3%2F%2F%2BQE%3D","line":8,"updated":"2013-12-25 23:46:07.000000000","message":"https://wiki.openstack.org/wiki/Gerrit_Workflow\n\nMissing the patch descriptions here.","commit_id":"8163bf4674b4e9664086d612e2fc540fe62d6549"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"c2763058aa77bdc6cf00dd136da3785055d01910","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2013-12-26 01:20:29 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix wrong message of MACAlreadyExists"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #1264127"},{"line_number":10,"context_line":"Change-Id: I315aa16fd82485b13d8d3bda56a2e0d1f368b937"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAT3%2F%2F%2BPk%3D","line":8,"in_reply_to":"AAAAT3%2F%2F%2BQE%3D","updated":"2013-12-26 00:28:12.000000000","message":"Are you think it is still resonable to add description for such simple fix?\n\nI will upload a short description if any other give -1 review","commit_id":"8163bf4674b4e9664086d612e2fc540fe62d6549"},{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"351289953fec19e0993aace2b0f39abe57308fc6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix wrong message of MACAlreadyExists"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #1264127"},{"line_number":10,"context_line":"Change-Id: I315aa16fd82485b13d8d3bda56a2e0d1f368b937"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAT3%2F%2F%2BQA%3D","line":10,"updated":"2013-12-25 23:46:07.000000000","message":"The order should be:\n\nThe patch title\nThe descriptions of this patch\nChange-Id: xxx\nCloses-Bug: #xxx","commit_id":"8163bf4674b4e9664086d612e2fc540fe62d6549"},{"author":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"change_message_id":"f6090fa2bb2d8f7739f44336e3909ff553b02328","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix wrong message of MACAlreadyExists"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #1264127"},{"line_number":10,"context_line":"Change-Id: I315aa16fd82485b13d8d3bda56a2e0d1f368b937"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAT3%2F%2F8YU%3D","line":10,"in_reply_to":"AAAAT3%2F%2F%2BPM%3D","updated":"2013-12-26 16:07:15.000000000","message":"I agree with ZhiQiang Fan. A common practice is to put Change-id as the last line to allow viewers to catch it with their eyes easily.","commit_id":"8163bf4674b4e9664086d612e2fc540fe62d6549"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"c2763058aa77bdc6cf00dd136da3785055d01910","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix wrong message of MACAlreadyExists"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #1264127"},{"line_number":10,"context_line":"Change-Id: I315aa16fd82485b13d8d3bda56a2e0d1f368b937"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAT3%2F%2F%2BPM%3D","line":10,"in_reply_to":"AAAAT3%2F%2F%2BQA%3D","updated":"2013-12-26 00:28:12.000000000","message":"Is there any reference for this rule?\n\nActually, there are some reviewers in other project recommend me to put closes-bug in front of change-id. And other reviewers think both are ok.\n\nThe Change-Id is appended automatically when we use git commit.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages, please read the good example part.","commit_id":"8163bf4674b4e9664086d612e2fc540fe62d6549"}]}