)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"32d53fc91f203ca1548512eba177b1cdf89b7eb6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added check for empty attributes in trove CLI"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Validated that new attributes are passed in for user-update-attributes.  Added unit test and changed help message."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I1363278912fb5ffddd2e64550dc6ba70df8a9df8"},{"line_number":12,"context_line":"Closes-Bug: 1298760"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9ad9bd40_7ba22d9a","line":9,"updated":"2014-05-06 08:51:36.000000000","message":"Please wrap commit message lines at 72 chars.\n\nOpenStack uses a common git commit message structure across projects. \n\nDetails at: https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_GIT_commit_message_structure","commit_id":"a0dad59018f84a646c03de9dc3d84c93feb48011"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"d96f2240489e725a0a04ae1d8a94a5a56746adb4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added check for empty attributes in trove CLI"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Validated that new attributes are passed in for user-update-attributes.  Added unit test and changed help message."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I1363278912fb5ffddd2e64550dc6ba70df8a9df8"},{"line_number":12,"context_line":"Closes-Bug: 1298760"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9ad9bd40_af1b8a71","line":9,"in_reply_to":"9ad9bd40_7ba22d9a","updated":"2014-05-06 14:00:29.000000000","message":"Sorry, for some reason I thought this happened automatically here.  Oops!","commit_id":"a0dad59018f84a646c03de9dc3d84c93feb48011"}],"troveclient/v1/users.py":[{"author":{"_account_id":6162,"name":"Dan Nguyen","email":"dan.nguyens.mail@gmail.com","username":"daniel-a-nguyen"},"change_message_id":"44391b3ed7486cb3442ba6156c53d781583b686c","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        user \u003d common.quote_user_host(username, hostname)"},{"line_number":80,"context_line":"        user_dict \u003d {}"},{"line_number":81,"context_line":"        if not newuserattr:"},{"line_number":82,"context_line":"            raise Exception(\"No updates specified for user %s\" % username)"},{"line_number":83,"context_line":"        user_dict[\u0027user\u0027] \u003d newuserattr"},{"line_number":84,"context_line":"        url \u003d \"/instances/%s/users/%s\" % (instance_id, user)"},{"line_number":85,"context_line":"        resp, body \u003d self.api.client.put(url, body\u003duser_dict)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9ad9bd40_2286c313","line":82,"updated":"2014-05-06 00:17:16.000000000","message":"Maybe move this check up as the first statement in the method since we raise an Exception if the test evaluates to True.","commit_id":"a0dad59018f84a646c03de9dc3d84c93feb48011"},{"author":{"_account_id":11316,"name":"Steve Leon","email":"kokhang@gmail.com","username":"kokhang"},"change_message_id":"ba40b5d4c3c96fae1399a6e5d61da6ebea1e1118","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        :rtype: :class:`User`."},{"line_number":77,"context_line":"        \"\"\""},{"line_number":78,"context_line":"        if not newuserattr:"},{"line_number":79,"context_line":"            raise Exception(\"No updates specified for user %s\" % username)"},{"line_number":80,"context_line":"        instance_id \u003d base.getid(instance)"},{"line_number":81,"context_line":"        user \u003d common.quote_user_host(username, hostname)"},{"line_number":82,"context_line":"        user_dict \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"7adec928_e85a7c4f","line":79,"updated":"2014-05-07 17:44:34.000000000","message":"I know this is not used consistently in the client but it might make sense to wrap your string in _(msg) (common.gettextutils.py) for localization. I am not sure if this is required for the client so i wont -1 this.","commit_id":"184c46c90e9b248ed6f3faed37c9aa5ee64408e2"},{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"6e8c44ec4f0dac7000fa969bcebd94f40625c017","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        :rtype: :class:`User`."},{"line_number":77,"context_line":"        \"\"\""},{"line_number":78,"context_line":"        if not newuserattr:"},{"line_number":79,"context_line":"            raise Exception(\"No updates specified for user %s\" % username)"},{"line_number":80,"context_line":"        instance_id \u003d base.getid(instance)"},{"line_number":81,"context_line":"        user \u003d common.quote_user_host(username, hostname)"},{"line_number":82,"context_line":"        user_dict \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"7adec928_8a2a937c","line":79,"in_reply_to":"7adec928_cf94e246","updated":"2014-05-09 19:10:16.000000000","message":"agreed...","commit_id":"184c46c90e9b248ed6f3faed37c9aa5ee64408e2"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"ea0b94d9884c612e666bf3d0cec0f03d9ba9d954","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        :rtype: :class:`User`."},{"line_number":77,"context_line":"        \"\"\""},{"line_number":78,"context_line":"        if not newuserattr:"},{"line_number":79,"context_line":"            raise Exception(\"No updates specified for user %s\" % username)"},{"line_number":80,"context_line":"        instance_id \u003d base.getid(instance)"},{"line_number":81,"context_line":"        user \u003d common.quote_user_host(username, hostname)"},{"line_number":82,"context_line":"        user_dict \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"7adec928_cf94e246","line":79,"in_reply_to":"7adec928_e85a7c4f","updated":"2014-05-07 20:40:33.000000000","message":"I did a quick check and it seems like no localization has been done for the client at all.  Maybe it should be a separate bug?","commit_id":"184c46c90e9b248ed6f3faed37c9aa5ee64408e2"}]}
