)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"7c99f0f10ddb6f6b96fd87099a29fefcb2224992","unresolved":false,"context_lines":[{"line_number":12,"context_line":"add a similar helper to add --project option to a"},{"line_number":13,"context_line":"parser."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This patch only use this helper in network because"},{"line_number":16,"context_line":"they are all the same. In other files, there are"},{"line_number":17,"context_line":"also --project options with different help message."},{"line_number":18,"context_line":"Will deal with them in the future."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa0719c6_7fae0382","line":15,"updated":"2016-03-23 08:52:35.000000000","message":"uses","commit_id":"a49ba4e7374af67de7d2f8e9c75842e173fd733a"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"ad3c1b4f09ddcbc444549c73b77f6d2458c1ba7b","unresolved":false,"context_lines":[{"line_number":12,"context_line":"add a similar helper to add --project option to a"},{"line_number":13,"context_line":"parser."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This patch only use this helper in network because"},{"line_number":16,"context_line":"they are all the same. In other files, there are"},{"line_number":17,"context_line":"also --project options with different help message."},{"line_number":18,"context_line":"Will deal with them in the future."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa0719c6_b4f71d8a","line":15,"in_reply_to":"fa0719c6_7fae0382","updated":"2016-03-23 09:29:26.000000000","message":"Done","commit_id":"a49ba4e7374af67de7d2f8e9c75842e173fd733a"}],"openstackclient/identity/common.py":[{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"49c246b63b0789a9147a2e0402682a7db8f92075","unresolved":false,"context_lines":[{"line_number":152,"context_line":"    parser.add_argument("},{"line_number":153,"context_line":"        \u0027--project\u0027,"},{"line_number":154,"context_line":"        metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":155,"context_line":"        help\u003d(\"Owner\u0027s project (name or ID).\")"},{"line_number":156,"context_line":"    )"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa0719c6_6af1ce6a","line":155,"updated":"2016-03-24 12:59:21.000000000","message":"Minor comment, but some use period at the end and others not...I think you\u0027ll need to adjust the doc accordingly.","commit_id":"95d7cbe2d092bd1d62e7a9aafed5178bde9b66ee"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"b4a9b534d5c079941033a837488e7f9089bbfc5d","unresolved":false,"context_lines":[{"line_number":152,"context_line":"    parser.add_argument("},{"line_number":153,"context_line":"        \u0027--project\u0027,"},{"line_number":154,"context_line":"        metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":155,"context_line":"        help\u003d(\"Owner\u0027s project (name or ID).\")"},{"line_number":156,"context_line":"    )"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a18252c_f1990d21","line":155,"in_reply_to":"5a18252c_8a6237d7","updated":"2016-04-12 13:15:29.000000000","message":"Correct, that seems to the be general rule.  You could propose a devref to make it official.","commit_id":"95d7cbe2d092bd1d62e7a9aafed5178bde9b66ee"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"c9baabc6fb1d06d12d3b1e01ab8f88d8e29a1102","unresolved":false,"context_lines":[{"line_number":152,"context_line":"    parser.add_argument("},{"line_number":153,"context_line":"        \u0027--project\u0027,"},{"line_number":154,"context_line":"        metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":155,"context_line":"        help\u003d(\"Owner\u0027s project (name or ID).\")"},{"line_number":156,"context_line":"    )"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a18252c_a9e4573b","line":155,"in_reply_to":"5a18252c_8a6237d7","updated":"2016-04-12 09:17:21.000000000","message":"Shall we all punctuations for all docs and comments ?\n\nI mean adding the punctuations in docs and comments could be a rule, not mean to make a big patch to do so.","commit_id":"95d7cbe2d092bd1d62e7a9aafed5178bde9b66ee"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"b4a9b534d5c079941033a837488e7f9089bbfc5d","unresolved":false,"context_lines":[{"line_number":152,"context_line":"    parser.add_argument("},{"line_number":153,"context_line":"        \u0027--project\u0027,"},{"line_number":154,"context_line":"        metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":155,"context_line":"        help\u003d(\"Owner\u0027s project (name or ID).\")"},{"line_number":156,"context_line":"    )"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a18252c_b17e75ac","line":155,"in_reply_to":"5a18252c_a9e4573b","updated":"2016-04-12 13:15:29.000000000","message":"In this case, I think the period should be removed from the help text.  As for a general punctuation cleanup of help text and documentation, I suggest getting community consensus via a devref before proceeding with that work.","commit_id":"95d7cbe2d092bd1d62e7a9aafed5178bde9b66ee"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"65183ba9efd6ea45c773535c980dc4f6bd28ff83","unresolved":false,"context_lines":[{"line_number":152,"context_line":"    parser.add_argument("},{"line_number":153,"context_line":"        \u0027--project\u0027,"},{"line_number":154,"context_line":"        metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":155,"context_line":"        help\u003d(\"Owner\u0027s project (name or ID).\")"},{"line_number":156,"context_line":"    )"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a18252c_8a6237d7","line":155,"in_reply_to":"fa0719c6_6af1ce6a","updated":"2016-04-12 02:05:26.000000000","message":"Yes, about this, shall we make some rules for it ?\n\nIf the comment has only one sentence, I often remove the period. But sometimes, the comment has two sentences. Will we add a period in the end ?","commit_id":"95d7cbe2d092bd1d62e7a9aafed5178bde9b66ee"}]}