)]}'
{"openstackclient/network/v2/port.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"68561089e1e1a90295e03fb38255313b03b0be98","unresolved":false,"context_lines":[{"line_number":144,"context_line":"        parser.add_argument("},{"line_number":145,"context_line":"            \u0027--device-id\u0027,"},{"line_number":146,"context_line":"            metavar\u003d\u0027\u003cdevice-id\u003e\u0027,"},{"line_number":147,"context_line":"            help\u003dargparse.SUPPRESS,"},{"line_number":148,"context_line":"        )"},{"line_number":149,"context_line":"        parser.add_argument("},{"line_number":150,"context_line":"            \u0027--device-owner\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afc51ec_4313b866","line":147,"updated":"2016-03-14 06:58:31.000000000","message":"When a user would be checking Help, then device-id would not be visible, only device would be visible.\nHowever, when user executes a CLI with the device-id parameter, it would work.\n\nMy point are:\na) instead of suppress, can we specify it as Deprecated? ( For me argparse.SUPPRESS works as good as a message to the user showing that the message is deprecated. But still wanted to know what your thoughts are about this)\nb) Should we have a check if user provides both device and device-id ( I am not in favor of this check, but would still like to know your opinion)","commit_id":"16941f0b23f2f6b031c4be2b143e839c5b954893"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"9c1861f2e990209ea6fe49ec5bdb3438264ae083","unresolved":false,"context_lines":[{"line_number":144,"context_line":"        parser.add_argument("},{"line_number":145,"context_line":"            \u0027--device-id\u0027,"},{"line_number":146,"context_line":"            metavar\u003d\u0027\u003cdevice-id\u003e\u0027,"},{"line_number":147,"context_line":"            help\u003dargparse.SUPPRESS,"},{"line_number":148,"context_line":"        )"},{"line_number":149,"context_line":"        parser.add_argument("},{"line_number":150,"context_line":"            \u0027--device-owner\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afc51ec_e5a61352","line":147,"in_reply_to":"3afc51ec_046fcf2b","updated":"2016-03-15 02:03:52.000000000","message":"I agree that a help test should be sufficient enough.","commit_id":"16941f0b23f2f6b031c4be2b143e839c5b954893"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"bccdb064779290571fe28fc4c425a0daf2e88dd5","unresolved":false,"context_lines":[{"line_number":144,"context_line":"        parser.add_argument("},{"line_number":145,"context_line":"            \u0027--device-id\u0027,"},{"line_number":146,"context_line":"            metavar\u003d\u0027\u003cdevice-id\u003e\u0027,"},{"line_number":147,"context_line":"            help\u003dargparse.SUPPRESS,"},{"line_number":148,"context_line":"        )"},{"line_number":149,"context_line":"        parser.add_argument("},{"line_number":150,"context_line":"            \u0027--device-owner\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3afc51ec_046fcf2b","line":147,"in_reply_to":"3afc51ec_4313b866","updated":"2016-03-14 19:10:01.000000000","message":"The only reason for maintaining the old option is back-compat.  We have historically renamed options in this manner, especially a new-ish one that has little history in the field.  Had it not been released I wouldn\u0027t even do this much, it never should have had this name to begin with.\n\nIf both are specified, the last one wins.  They could go into an exclusive group, I think the help text would do the right thing.  But I am not convinced this is necessary.","commit_id":"16941f0b23f2f6b031c4be2b143e839c5b954893"}]}