)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"96d93186977afa218d5e1e79d84d87a5c911190f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fa65d878_3eae386f","updated":"2022-08-22 19:04:50.000000000","message":"Please add unit and functional tests also:\nUnit tests: https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/tests/unit/network/v2/test_network.py\n\nFunctional tests: https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/tests/functional/network/v2/test_network.py","commit_id":"138ef0f682bbc9a035015b905788acc4e0b7dd66"}],"openstackclient/network/v2/network.py":[{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"c3443ae7eafe3cda76963b602e95773e0333ca22","unresolved":false,"context_lines":[{"line_number":368,"context_line":"            nargs\u003d\"+\","},{"line_number":369,"context_line":"            help\u003d_(\"Network(s) to delete (name or ID)\")"},{"line_number":370,"context_line":"        )"},{"line_number":371,"context_line":"        parse.add_argument("},{"line_number":372,"context_line":"            \u0027--cascade\u0027,"},{"line_number":373,"context_line":"            metavar\u003d\"\u003ccascade\u003e\","},{"line_number":374,"context_line":"            help\u003dself.enhance_help_neutron("}],"source_content_type":"text/x-python","patch_set":1,"id":"8d8f5b77_073a2845","line":371,"in_reply_to":"4cc0c656_7106ceba","updated":"2022-08-23 07:54:33.000000000","message":"Done","commit_id":"138ef0f682bbc9a035015b905788acc4e0b7dd66"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"96d93186977afa218d5e1e79d84d87a5c911190f","unresolved":true,"context_lines":[{"line_number":368,"context_line":"            nargs\u003d\"+\","},{"line_number":369,"context_line":"            help\u003d_(\"Network(s) to delete (name or ID)\")"},{"line_number":370,"context_line":"        )"},{"line_number":371,"context_line":"        parse.add_argument("},{"line_number":372,"context_line":"            \u0027--cascade\u0027,"},{"line_number":373,"context_line":"            metavar\u003d\"\u003ccascade\u003e\","},{"line_number":374,"context_line":"            help\u003dself.enhance_help_neutron("}],"source_content_type":"text/x-python","patch_set":1,"id":"4cc0c656_7106ceba","line":371,"in_reply_to":"e565c301_f58c8d78","updated":"2022-08-22 19:04:50.000000000","message":"\u003e pep8: F821 undefined name \u0027parse\u0027\n\nPlease fix pep8 errors, as I see this error is causing the failure of unit tests also.","commit_id":"138ef0f682bbc9a035015b905788acc4e0b7dd66"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"91f34166f0d0a7cffd99de1dc13c604411d21912","unresolved":true,"context_lines":[{"line_number":369,"context_line":"            help\u003d_(\"Network(s) to delete (name or ID)\")"},{"line_number":370,"context_line":"        )"},{"line_number":371,"context_line":"        parser.add_argument("},{"line_number":372,"context_line":"            \u0027--cascade\u0027,"},{"line_number":373,"context_line":"            metavar\u003d\"\u003ccascade\u003e\","},{"line_number":374,"context_line":"            help\u003dself.enhance_help_neutron("},{"line_number":375,"context_line":"                _(\"Delete network(s) and all associated resources\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"5e7ff153_5cf061d4","line":372,"range":{"start_line":372,"start_character":15,"end_line":372,"end_character":22},"updated":"2022-08-24 13:29:28.000000000","message":"to be consequent with other boolean options this should be a pair: cascade and no-cascade.\n\nFor example see the ListNetwork and --external and --internal how to solve it. You need a mutually_exclusive_group for it to avoid such situation when somebody added both --cascade and --no-cascade.\n\nYou have to take care of these in take_action_network method, see https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/network/v2/network.py#L549-L554","commit_id":"826b0b2f6528dcdee63f53673abffc1084aac804"}]}