)]}'
{"doc/source/command-objects/network.rst":[{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"e8d560d7f1a70de7a6beceaaef94ba1934081ad3","unresolved":false,"context_lines":[{"line_number":190,"context_line":"        [--project]"},{"line_number":191,"context_line":"        [--share]"},{"line_number":192,"context_line":"        [--no-share]"},{"line_number":193,"context_line":"        [--status]"},{"line_number":194,"context_line":""},{"line_number":195,"context_line":".. option:: --external"},{"line_number":196,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"bacf61ea_ed71cf45","line":193,"range":{"start_line":193,"start_character":8,"end_line":193,"end_character":18},"updated":"2016-08-04 09:28:50.000000000","message":"this not a flag option. should be like this:\n[--status \u003cstatus\u003e]","commit_id":"3b9317a86696d0c7146ddc035d005ff5fccbd232"},{"author":{"_account_id":20079,"name":"Ha Van Tu","email":"tuhv@vn.fujitsu.com","username":"tuhv"},"change_message_id":"869083bd0d8864753534421514d3847dffad2741","unresolved":false,"context_lines":[{"line_number":190,"context_line":"        [--project]"},{"line_number":191,"context_line":"        [--share]"},{"line_number":192,"context_line":"        [--no-share]"},{"line_number":193,"context_line":"        [--status]"},{"line_number":194,"context_line":""},{"line_number":195,"context_line":".. option:: --external"},{"line_number":196,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"bacf61ea_0d78a3f1","line":193,"range":{"start_line":193,"start_character":8,"end_line":193,"end_character":18},"in_reply_to":"bacf61ea_ed71cf45","updated":"2016-08-04 09:34:14.000000000","message":"Done","commit_id":"3b9317a86696d0c7146ddc035d005ff5fccbd232"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"d456b8f778bb4422d1432fbcb1320eac6904e07b","unresolved":false,"context_lines":[{"line_number":232,"context_line":""},{"line_number":233,"context_line":".. option:: --status \u003cstatus\u003e"},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"    List networks according to their status"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"network set"},{"line_number":238,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3ac371cc_e7093fdf","line":235,"updated":"2016-08-15 11:28:40.000000000","message":"This should match the help message","commit_id":"d187753cb305cfa01098d9f187aa65cfc6a98a22"},{"author":{"_account_id":20079,"name":"Ha Van Tu","email":"tuhv@vn.fujitsu.com","username":"tuhv"},"change_message_id":"2ef5a96f58450eb7cfb6a765e76ccf290567f2aa","unresolved":false,"context_lines":[{"line_number":232,"context_line":""},{"line_number":233,"context_line":".. option:: --status \u003cstatus\u003e"},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"    List networks according to their status"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"network set"},{"line_number":238,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3ac371cc_02f25b6f","line":235,"in_reply_to":"3ac371cc_e7093fdf","updated":"2016-08-16 07:11:43.000000000","message":"Done","commit_id":"d187753cb305cfa01098d9f187aa65cfc6a98a22"}],"openstackclient/network/v2/network.py":[{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"e8d560d7f1a70de7a6beceaaef94ba1934081ad3","unresolved":false,"context_lines":[{"line_number":350,"context_line":"            help\u003d_(\"List not shared networks\")"},{"line_number":351,"context_line":"        )"},{"line_number":352,"context_line":"        parser.add_argument("},{"line_number":353,"context_line":"            \u0027--status\u0027,"},{"line_number":354,"context_line":"            metavar\u003d\u0027\u003cstatus\u003e\u0027,"},{"line_number":355,"context_line":"            help\u003d_(\"List networks according to their status\")"},{"line_number":356,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_4dd3bb38","line":353,"updated":"2016-08-04 09:28:50.000000000","message":"Do we need add some status choices for users here?","commit_id":"3b9317a86696d0c7146ddc035d005ff5fccbd232"},{"author":{"_account_id":20079,"name":"Ha Van Tu","email":"tuhv@vn.fujitsu.com","username":"tuhv"},"change_message_id":"869083bd0d8864753534421514d3847dffad2741","unresolved":false,"context_lines":[{"line_number":350,"context_line":"            help\u003d_(\"List not shared networks\")"},{"line_number":351,"context_line":"        )"},{"line_number":352,"context_line":"        parser.add_argument("},{"line_number":353,"context_line":"            \u0027--status\u0027,"},{"line_number":354,"context_line":"            metavar\u003d\u0027\u003cstatus\u003e\u0027,"},{"line_number":355,"context_line":"            help\u003d_(\"List networks according to their status\")"},{"line_number":356,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_6da2ff4c","line":353,"in_reply_to":"bacf61ea_4dd3bb38","updated":"2016-08-04 09:34:14.000000000","message":"@Huanxuan Ao:\nThere should be \u0027ACTIVE\u0027, and \u0027ERROR\u0027, but it is not an enum type, it is a string type. So place it will make user confuse between these two types. How do you think?","commit_id":"3b9317a86696d0c7146ddc035d005ff5fccbd232"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"a443d4c953cc4d755071f494ae0e1f1bbe79625d","unresolved":false,"context_lines":[{"line_number":350,"context_line":"            help\u003d_(\"List not shared networks\")"},{"line_number":351,"context_line":"        )"},{"line_number":352,"context_line":"        parser.add_argument("},{"line_number":353,"context_line":"            \u0027--status\u0027,"},{"line_number":354,"context_line":"            metavar\u003d\u0027\u003cstatus\u003e\u0027,"},{"line_number":355,"context_line":"            help\u003d_(\"List networks according to their status\")"},{"line_number":356,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":3,"id":"bacf61ea_a7cfe454","line":353,"updated":"2016-08-04 14:15:45.000000000","message":"I wonder if a list of choices is appropriate here.","commit_id":"da6e18fd726e3a6a3af76e5d17281de32b07b677"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"d456b8f778bb4422d1432fbcb1320eac6904e07b","unresolved":false,"context_lines":[{"line_number":348,"context_line":"        )"},{"line_number":349,"context_line":"        parser.add_argument("},{"line_number":350,"context_line":"            \u0027--status\u0027,"},{"line_number":351,"context_line":"            metavar\u003d\u0027\u003cstatus\u003e\u0027,"},{"line_number":352,"context_line":"            help\u003d_(\"List networks according to their status\""},{"line_number":353,"context_line":"                   \"(\u0027ACTIVE\u0027, \u0027BUILD\u0027, \u0027DOWN\u0027, \u0027ERROR\u0027)\")"},{"line_number":354,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":6,"id":"3ac371cc_a71747ec","line":351,"updated":"2016-08-15 11:28:40.000000000","message":"You can add choices for this option, a example: http://docs.openstack.org/developer/python-openstackclient/command-options.html#options-with-choices","commit_id":"d187753cb305cfa01098d9f187aa65cfc6a98a22"},{"author":{"_account_id":20079,"name":"Ha Van Tu","email":"tuhv@vn.fujitsu.com","username":"tuhv"},"change_message_id":"2ef5a96f58450eb7cfb6a765e76ccf290567f2aa","unresolved":false,"context_lines":[{"line_number":348,"context_line":"        )"},{"line_number":349,"context_line":"        parser.add_argument("},{"line_number":350,"context_line":"            \u0027--status\u0027,"},{"line_number":351,"context_line":"            metavar\u003d\u0027\u003cstatus\u003e\u0027,"},{"line_number":352,"context_line":"            help\u003d_(\"List networks according to their status\""},{"line_number":353,"context_line":"                   \"(\u0027ACTIVE\u0027, \u0027BUILD\u0027, \u0027DOWN\u0027, \u0027ERROR\u0027)\")"},{"line_number":354,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":6,"id":"3ac371cc_a2038741","line":351,"in_reply_to":"3ac371cc_a71747ec","updated":"2016-08-16 07:11:43.000000000","message":"Done","commit_id":"d187753cb305cfa01098d9f187aa65cfc6a98a22"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"87df60449becc463ccd28ad84c44ffdf78df5a49","unresolved":false,"context_lines":[{"line_number":350,"context_line":"            \u0027--status\u0027,"},{"line_number":351,"context_line":"            metavar\u003d\u0027\u003cstatus\u003e\u0027,"},{"line_number":352,"context_line":"            choices\u003d[\u0027ACTIVE\u0027, \u0027BUILD\u0027, \u0027DOWN\u0027, \u0027ERROR\u0027],"},{"line_number":353,"context_line":"            help\u003d_(\"List networks according to their status\""},{"line_number":354,"context_line":"                   \"(\u0027ACTIVE\u0027, \u0027BUILD\u0027, \u0027DOWN\u0027, \u0027ERROR\u0027)\")"},{"line_number":355,"context_line":"        )"},{"line_number":356,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":7,"id":"3ac371cc_c98027fa","line":353,"range":{"start_line":353,"start_character":58,"end_line":353,"end_character":59},"updated":"2016-08-17 08:37:27.000000000","message":"You need a trailing blank space here.","commit_id":"4228651429e48eec7af04b2d831b371246787df9"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"c4a2448361d13044100cd5fce8ba039e0a062df3","unresolved":false,"context_lines":[{"line_number":350,"context_line":"            \u0027--status\u0027,"},{"line_number":351,"context_line":"            metavar\u003d\u0027\u003cstatus\u003e\u0027,"},{"line_number":352,"context_line":"            choices\u003d[\u0027ACTIVE\u0027, \u0027BUILD\u0027, \u0027DOWN\u0027, \u0027ERROR\u0027],"},{"line_number":353,"context_line":"            help\u003d_(\"List networks according to their status\""},{"line_number":354,"context_line":"                   \"(\u0027ACTIVE\u0027, \u0027BUILD\u0027, \u0027DOWN\u0027, \u0027ERROR\u0027)\")"},{"line_number":355,"context_line":"        )"},{"line_number":356,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":8,"id":"3ac371cc_98182920","line":353,"range":{"start_line":353,"start_character":53,"end_line":353,"end_character":59},"updated":"2016-08-17 12:58:00.000000000","message":"Please add a space here.","commit_id":"1347b669b0977536e3dd72b0d54f9e4e80fc978c"}]}