summaryrefslogtreecommitdiff
path: root/gitlab/v4/cli.py
diff options
context:
space:
mode:
authorJohn L. Villalovos <john@sodarock.com>2022-06-03 21:21:38 -0700
committerJohn L. Villalovos <john@sodarock.com>2022-06-03 21:24:58 -0700
commitc86e471dead930468172f4b7439ea6fa207f12e8 (patch)
treeb62a4f0651ccf75ecd490a2512090f4838bf6e2b /gitlab/v4/cli.py
parent6189437d2c8d18f6c7d72aa7743abd6d36fb4efa (diff)
downloadgitlab-c86e471dead930468172f4b7439ea6fa207f12e8.tar.gz
chore: rename `what` to `gitlab_resource`
Naming a variable `what` makes it difficult to understand what it is used for. Rename it to `gitlab_resource` as that is what is being stored. The Gitlab documentation talks about them being resources: https://docs.gitlab.com/ee/api/api_resources.html This will improve code readability.
Diffstat (limited to 'gitlab/v4/cli.py')
-rw-r--r--gitlab/v4/cli.py18
1 files changed, 9 insertions, 9 deletions
diff --git a/gitlab/v4/cli.py b/gitlab/v4/cli.py
index 226459e..18bd3f0 100644
--- a/gitlab/v4/cli.py
+++ b/gitlab/v4/cli.py
@@ -29,13 +29,13 @@ from gitlab import cli
class GitlabCLI:
def __init__(
- self, gl: gitlab.Gitlab, what: str, action: str, args: Dict[str, str]
+ self, gl: gitlab.Gitlab, gitlab_resource: str, action: str, args: Dict[str, str]
) -> None:
- self.cls: Type[gitlab.base.RESTObject] = cli.what_to_cls(
- what, namespace=gitlab.v4.objects
+ self.cls: Type[gitlab.base.RESTObject] = cli.gitlab_resource_to_cls(
+ gitlab_resource, namespace=gitlab.v4.objects
)
self.cls_name = self.cls.__name__
- self.what = what.replace("-", "_")
+ self.gitlab_resource = gitlab_resource.replace("-", "_")
self.action = action.lower()
self.gl = gl
self.args = args
@@ -81,7 +81,7 @@ class GitlabCLI:
def run(self) -> Any:
# Check for a method that matches object + action
- method = f"do_{self.what}_{self.action}"
+ method = f"do_{self.gitlab_resource}_{self.action}"
if hasattr(self, method):
return getattr(self, method)()
@@ -333,7 +333,7 @@ def _populate_sub_parser_by_class(
def extend_parser(parser: argparse.ArgumentParser) -> argparse.ArgumentParser:
subparsers = parser.add_subparsers(
- title="object", dest="what", help="Object to manipulate."
+ title="object", dest="gitlab_resource", help="Object to manipulate."
)
subparsers.required = True
@@ -347,7 +347,7 @@ def extend_parser(parser: argparse.ArgumentParser) -> argparse.ArgumentParser:
classes.add(cls._obj_cls)
for cls in sorted(classes, key=operator.attrgetter("__name__")):
- arg_name = cli.cls_to_what(cls)
+ arg_name = cli.cls_to_gitlab_resource(cls)
object_group = subparsers.add_parser(arg_name)
object_subparsers = object_group.add_subparsers(
@@ -497,14 +497,14 @@ PRINTERS: Dict[
def run(
gl: gitlab.Gitlab,
- what: str,
+ gitlab_resource: str,
action: str,
args: Dict[str, Any],
verbose: bool,
output: str,
fields: List[str],
) -> None:
- g_cli = GitlabCLI(gl=gl, what=what, action=action, args=args)
+ g_cli = GitlabCLI(gl=gl, gitlab_resource=gitlab_resource, action=action, args=args)
data = g_cli.run()
printer: Union[JSONPrinter, LegacyPrinter, YAMLPrinter] = PRINTERS[output]()