Merge pull request #1738 from python-gitlab/jlvillal/pylint_fixes

nejch · web-flow · commit 3679591adaba · 2021-12-08T08:31:19.000+01:00
chore: fix pylint error "expression-not-assigned"
diff --git a/gitlab/v4/cli.py b/gitlab/v4/cli.py
@@ -266,20 +266,16 @@ def _populate_sub_parser_by_class(
                     sub_parser_action.add_argument(f"--{id_attr}", required=True)
 
             required, optional, dummy = cli.custom_actions[name][action_name]
-            [
-                sub_parser_action.add_argument(
-                    f"--{x.replace('_', '-')}", required=True
-                )
-                for x in required
-                if x != cls._id_attr
-            ]
-            [
-                sub_parser_action.add_argument(
-                    f"--{x.replace('_', '-')}", required=False
-                )
-                for x in optional
-                if x != cls._id_attr
-            ]
+            for x in required:
+                if x != cls._id_attr:
+                    sub_parser_action.add_argument(
+                        f"--{x.replace('_', '-')}", required=True
+                    )
+            for x in optional:
+                if x != cls._id_attr:
+                    sub_parser_action.add_argument(
+                        f"--{x.replace('_', '-')}", required=False
+                    )
 
     if mgr_cls.__name__ in cli.custom_actions:
         name = mgr_cls.__name__
@@ -293,20 +289,16 @@ def _populate_sub_parser_by_class(
                 sub_parser_action.add_argument("--sudo", required=False)
 
             required, optional, dummy = cli.custom_actions[name][action_name]
-            [
-                sub_parser_action.add_argument(
-                    f"--{x.replace('_', '-')}", required=True
-                )
-                for x in required
-                if x != cls._id_attr
-            ]
-            [
-                sub_parser_action.add_argument(
-                    f"--{x.replace('_', '-')}", required=False
-                )
-                for x in optional
-                if x != cls._id_attr
-            ]
+            for x in required:
+                if x != cls._id_attr:
+                    sub_parser_action.add_argument(
+                        f"--{x.replace('_', '-')}", required=True
+                    )
+            for x in optional:
+                if x != cls._id_attr:
+                    sub_parser_action.add_argument(
+                        f"--{x.replace('_', '-')}", required=False
+                    )
 
 
 def extend_parser(parser: argparse.ArgumentParser) -> argparse.ArgumentParser:
diff --git a/pyproject.toml b/pyproject.toml
@@ -55,7 +55,6 @@ disable = [
     "consider-using-sys-exit",
     "cyclic-import",
     "duplicate-code",
-    "expression-not-assigned",
     "fixme",
     "implicit-str-concat",
     "import-outside-toplevel",
