diff options
| author | John L. Villalovos <john@sodarock.com> | 2021-03-07 11:31:23 -0800 |
|---|---|---|
| committer | Nejc Habjan <hab.nejc@gmail.com> | 2021-04-17 17:09:41 +0200 |
| commit | aee1f496c1f414c1e30909767d53ae624fe875e7 (patch) | |
| tree | 04a7fe9d8f77cffeae733bb8dd3447911995f3a1 /gitlab/v4/cli.py | |
| parent | 8603248f73d8c751023fbfd2a394c5b7d939af7f (diff) | |
| download | gitlab-aee1f496c1f414c1e30909767d53ae624fe875e7.tar.gz | |
chore: have _create_attrs & _update_attrs be a namedtuple
Convert _create_attrs and _update_attrs to use a NamedTuple
(RequiredOptional) to help with code readability. Update all code to
use the NamedTuple.
Diffstat (limited to 'gitlab/v4/cli.py')
| -rw-r--r-- | gitlab/v4/cli.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/gitlab/v4/cli.py b/gitlab/v4/cli.py index df645bf..d036d12 100644 --- a/gitlab/v4/cli.py +++ b/gitlab/v4/cli.py @@ -177,11 +177,11 @@ def _populate_sub_parser_by_class(cls, sub_parser): ] if action_name == "create": - for x in mgr_cls._create_attrs[0]: + for x in mgr_cls._create_attrs.required: sub_parser_action.add_argument( "--%s" % x.replace("_", "-"), required=True ) - for x in mgr_cls._create_attrs[1]: + for x in mgr_cls._create_attrs.optional: sub_parser_action.add_argument( "--%s" % x.replace("_", "-"), required=False ) @@ -191,13 +191,13 @@ def _populate_sub_parser_by_class(cls, sub_parser): id_attr = cls._id_attr.replace("_", "-") sub_parser_action.add_argument("--%s" % id_attr, required=True) - for x in mgr_cls._update_attrs[0]: + for x in mgr_cls._update_attrs.required: if x != cls._id_attr: sub_parser_action.add_argument( "--%s" % x.replace("_", "-"), required=True ) - for x in mgr_cls._update_attrs[1]: + for x in mgr_cls._update_attrs.optional: if x != cls._id_attr: sub_parser_action.add_argument( "--%s" % x.replace("_", "-"), required=False |