)]}'
{"magnumclient/v1/clusters_shell.py":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"7aa90abe5f9eeb1944403511f0183723b4f3e312","unresolved":false,"context_lines":[{"line_number":93,"context_line":"           dest\u003d\u0027keypair\u0027,"},{"line_number":94,"context_line":"           metavar\u003d\u0027\u003ckeypair\u003e\u0027,"},{"line_number":95,"context_line":"           default\u003dNone,"},{"line_number":96,"context_line":"           help\u003dutils.deprecation_message("},{"line_number":97,"context_line":"               \u0027UUID or name of the keypair to use for this cluster.\u0027,"},{"line_number":98,"context_line":"               \u0027keypair\u0027))"},{"line_number":99,"context_line":"@utils.arg(\u0027--keypair\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_b9d2cd32","line":96,"updated":"2016-10-19 08:05:09.000000000","message":"I think, we can supress \"help\" for deprecated param. As i guess documentation will also be updated against this bp.\nSo new users are going to use new params, if users who uses old param they will get deprecation warning. So if cli users see help they get only recommended parameter to use.","commit_id":"d3cdbff66088c7303a260fd8435d7b94189067a2"},{"author":{"_account_id":21660,"name":"Stephen Watson","email":"stephen.watson@intel.com","username":"swatson"},"change_message_id":"81594efba1f0a4ae468ab1259e268c26a663fb17","unresolved":false,"context_lines":[{"line_number":93,"context_line":"           dest\u003d\u0027keypair\u0027,"},{"line_number":94,"context_line":"           metavar\u003d\u0027\u003ckeypair\u003e\u0027,"},{"line_number":95,"context_line":"           default\u003dNone,"},{"line_number":96,"context_line":"           help\u003dutils.deprecation_message("},{"line_number":97,"context_line":"               \u0027UUID or name of the keypair to use for this cluster.\u0027,"},{"line_number":98,"context_line":"               \u0027keypair\u0027))"},{"line_number":99,"context_line":"@utils.arg(\u0027--keypair\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_f3e1922b","line":96,"in_reply_to":"fa6399be_b9d2cd32","updated":"2016-10-20 07:32:15.000000000","message":"So leave the deprecated parameters completely out of the help text? I like the idea. \n\nI\u0027ll investigate how to go about doing this and hopefully get a good solution in the next patch","commit_id":"d3cdbff66088c7303a260fd8435d7b94189067a2"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"2d41e0b1e681f1d3d94447c9fa25474062313c21","unresolved":false,"context_lines":[{"line_number":93,"context_line":"           dest\u003d\u0027keypair\u0027,"},{"line_number":94,"context_line":"           metavar\u003d\u0027\u003ckeypair\u003e\u0027,"},{"line_number":95,"context_line":"           default\u003dNone,"},{"line_number":96,"context_line":"           help\u003dutils.deprecation_message("},{"line_number":97,"context_line":"               \u0027UUID or name of the keypair to use for this cluster.\u0027,"},{"line_number":98,"context_line":"               \u0027keypair\u0027))"},{"line_number":99,"context_line":"@utils.arg(\u0027--keypair\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_541ea170","line":96,"in_reply_to":"fa6399be_f3e1922b","updated":"2016-10-20 11:02:45.000000000","message":"You can try argparse.SUPPRESS","commit_id":"d3cdbff66088c7303a260fd8435d7b94189067a2"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"7aa90abe5f9eeb1944403511f0183723b4f3e312","unresolved":false,"context_lines":[{"line_number":100,"context_line":"           dest\u003d\u0027keypair\u0027,"},{"line_number":101,"context_line":"           metavar\u003d\u0027\u003ckeypair\u003e\u0027,"},{"line_number":102,"context_line":"           default\u003dNone,"},{"line_number":103,"context_line":"           help\u003d\u0027UUID Or name of the keypair to use for this cluster.\u0027)"},{"line_number":104,"context_line":"@utils.arg(\u0027--node-count\u0027,"},{"line_number":105,"context_line":"           metavar\u003d\u0027\u003cnode-count\u003e\u0027,"},{"line_number":106,"context_line":"           type\u003dint,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_59e29102","line":103,"range":{"start_line":103,"start_character":22,"end_line":103,"end_character":23},"updated":"2016-10-19 08:05:09.000000000","message":"Or --\u003e or","commit_id":"d3cdbff66088c7303a260fd8435d7b94189067a2"},{"author":{"_account_id":21660,"name":"Stephen Watson","email":"stephen.watson@intel.com","username":"swatson"},"change_message_id":"81594efba1f0a4ae468ab1259e268c26a663fb17","unresolved":false,"context_lines":[{"line_number":100,"context_line":"           dest\u003d\u0027keypair\u0027,"},{"line_number":101,"context_line":"           metavar\u003d\u0027\u003ckeypair\u003e\u0027,"},{"line_number":102,"context_line":"           default\u003dNone,"},{"line_number":103,"context_line":"           help\u003d\u0027UUID Or name of the keypair to use for this cluster.\u0027)"},{"line_number":104,"context_line":"@utils.arg(\u0027--node-count\u0027,"},{"line_number":105,"context_line":"           metavar\u003d\u0027\u003cnode-count\u003e\u0027,"},{"line_number":106,"context_line":"           type\u003dint,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_13d1a657","line":103,"range":{"start_line":103,"start_character":22,"end_line":103,"end_character":23},"in_reply_to":"fa6399be_59e29102","updated":"2016-10-20 07:32:15.000000000","message":"Good catch! Will fix in next patch","commit_id":"d3cdbff66088c7303a260fd8435d7b94189067a2"}]}