fix(cli): inform argument parser that options are mutually exclusive

Sjord · Sjord · commit a53397d75bae · 2024-01-28T21:36:56.000+01:00
diff --git a/gitlab/v4/cli.py b/gitlab/v4/cli.py
@@ -258,10 +258,14 @@ def _populate_sub_parser_by_class(
                 sub_parser_action.add_argument(
                     f"--{x.replace('_', '-')}", required=True
                 )
-            for x in mgr_cls._create_attrs.optional + mgr_cls._create_attrs.exclusive:
+            for x in mgr_cls._create_attrs.optional:
                 sub_parser_action.add_argument(
                     f"--{x.replace('_', '-')}", required=False
                 )
+            if mgr_cls._create_attrs.exclusive:
+                group = sub_parser_action.add_mutually_exclusive_group()
+                for x in mgr_cls._create_attrs.exclusive:
+                    group.add_argument(f"--{x.replace('_', '-')}")
 
         if action_name == "update":
             if cls._id_attr is not None:
@@ -274,12 +278,17 @@ def _populate_sub_parser_by_class(
                         f"--{x.replace('_', '-')}", required=True
                     )
 
-            for x in mgr_cls._update_attrs.optional + mgr_cls._update_attrs.exclusive:
+            for x in mgr_cls._update_attrs.optional:
                 if x != cls._id_attr:
                     sub_parser_action.add_argument(
                         f"--{x.replace('_', '-')}", required=False
                     )
 
+            if mgr_cls._create_attrs.exclusive:
+                group = sub_parser_action.add_mutually_exclusive_group()
+                for x in mgr_cls._create_attrs.exclusive:
+                    group.add_argument(f"--{x.replace('_', '-')}")
+
     if cls.__name__ in cli.custom_actions:
         name = cls.__name__
         for action_name in cli.custom_actions[name]:
