)]}'
{"openstackclient/volume/v1/snapshot.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"d37cdc8177df3b1512a75d110466a3e4b9247c99","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        parser.add_argument("},{"line_number":41,"context_line":"            \u0027--name\u0027,"},{"line_number":42,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":43,"context_line":"            default\u003dNone,"},{"line_number":44,"context_line":"            help\u003d\u0027Name of the snapshot\u0027,"},{"line_number":45,"context_line":"        )"},{"line_number":46,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F858%3D","line":43,"updated":"2013-03-05 14:27:14.000000000","message":"Should these string options have string defaults?","commit_id":"8bde55101c2b38462b458b35ed39994a21f14074"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ccacd8cd43b7e763ad54afe117c10adfa60aef0b","unresolved":false,"context_lines":[{"line_number":37,"context_line":"            \u0027volume\u0027,"},{"line_number":38,"context_line":"            metavar\u003d\u0027\u003cvolume\u003e\u0027,"},{"line_number":39,"context_line":"            help\u003d\u0027The name or ID of the volume to snapshot\u0027,"},{"line_number":40,"context_line":"        )"},{"line_number":41,"context_line":"        parser.add_argument("},{"line_number":42,"context_line":"            \u0027--name\u0027,"},{"line_number":43,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2F2b4%3D","line":40,"updated":"2013-03-08 21:06:53.000000000","message":"A few changes to be able to pick up names for volume and snapshot objects. I\u0027ll go through the type/volume and update them once this change set is in.","commit_id":"0c4929373e04c48638dc3facd47e39933cc9ac49"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ccacd8cd43b7e763ad54afe117c10adfa60aef0b","unresolved":false,"context_lines":[{"line_number":43,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":44,"context_line":"            required\u003dTrue,"},{"line_number":45,"context_line":"            help\u003d\u0027Name of the snapshot\u0027,"},{"line_number":46,"context_line":"        )"},{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--description\u0027,"},{"line_number":49,"context_line":"            metavar\u003d\u0027\u003cdescription\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2F2cQ%3D","line":46,"updated":"2013-03-08 21:06:53.000000000","message":"I made the name argument a required one, so as to match the others","commit_id":"0c4929373e04c48638dc3facd47e39933cc9ac49"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"5f8bdfcef4599857370603a95c9cbb5d1ae4ecc2","unresolved":false,"context_lines":[{"line_number":43,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":44,"context_line":"            required\u003dTrue,"},{"line_number":45,"context_line":"            help\u003d\u0027Name of the snapshot\u0027,"},{"line_number":46,"context_line":"        )"},{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--description\u0027,"},{"line_number":49,"context_line":"            metavar\u003d\u0027\u003cdescription\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2Fyk4%3D","line":46,"in_reply_to":"AAAAN3%2F%2F2cQ%3D","updated":"2013-03-12 15:11:07.000000000","message":"If it\u0027s required, shouldn\u0027t it be a positional argument instead of using the option syntax?","commit_id":"0c4929373e04c48638dc3facd47e39933cc9ac49"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"78c049b941298dadad63e6416a67426a3452303d","unresolved":false,"context_lines":[{"line_number":43,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":44,"context_line":"            required\u003dTrue,"},{"line_number":45,"context_line":"            help\u003d\u0027Name of the snapshot\u0027,"},{"line_number":46,"context_line":"        )"},{"line_number":47,"context_line":"        parser.add_argument("},{"line_number":48,"context_line":"            \u0027--description\u0027,"},{"line_number":49,"context_line":"            metavar\u003d\u0027\u003cdescription\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2Fxvw%3D","line":46,"in_reply_to":"AAAAN3%2F%2Fyk4%3D","updated":"2013-03-12 21:03:41.000000000","message":"I was making is consistent with other create commands (check out https://review.openstack.org/#/c/23323/). I am indifferent to using X positional arguments, or setting optional arguments to be required.","commit_id":"0c4929373e04c48638dc3facd47e39933cc9ac49"}]}