diff options
| author | Max Wittig <max.wittig.ch@gmail.com> | 2021-04-25 16:35:30 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-25 16:35:30 +0200 |
| commit | 62c75b5e637858f0e9ef7bed21a347bbd5e0b972 (patch) | |
| tree | 2474ef09c081ed00506900bd90d9fbe91ed256ad | |
| parent | 09522b356386f4e2ceef7e8c2604269e0682ed20 (diff) | |
| parent | 89331131b3337308bacb0c4013e80a4809f3952c (diff) | |
| download | gitlab-62c75b5e637858f0e9ef7bed21a347bbd5e0b972.tar.gz | |
Merge pull request #1411 from JohnVillalovos/jlvillal/list_filters
chore: make ListMixin._list_filters always present
| -rw-r--r-- | gitlab/mixins.py | 2 | ||||
| -rw-r--r-- | gitlab/v4/cli.py | 11 |
2 files changed, 6 insertions, 7 deletions
diff --git a/gitlab/mixins.py b/gitlab/mixins.py index a22fea4..4c3e46e 100644 --- a/gitlab/mixins.py +++ b/gitlab/mixins.py @@ -22,6 +22,7 @@ from typing import ( Dict, List, Optional, + Tuple, Type, TYPE_CHECKING, Union, @@ -186,6 +187,7 @@ class RefreshMixin(_RestObjectBase): class ListMixin(_RestManagerBase): _computed_path: Optional[str] _from_parent_attrs: Dict[str, Any] + _list_filters: Tuple[str, ...] = () _obj_cls: Optional[Type[base.RESTObject]] _parent: Optional[base.RESTObject] _parent_attrs: Dict[str, Any] diff --git a/gitlab/v4/cli.py b/gitlab/v4/cli.py index b03883e..76eb1cb 100644 --- a/gitlab/v4/cli.py +++ b/gitlab/v4/cli.py @@ -145,13 +145,10 @@ def _populate_sub_parser_by_class(cls, sub_parser): ) if action_name == "list": - if hasattr(mgr_cls, "_list_filters"): - [ - sub_parser_action.add_argument( - "--%s" % x.replace("_", "-"), required=False - ) - for x in mgr_cls._list_filters - ] + for x in mgr_cls._list_filters: + sub_parser_action.add_argument( + "--%s" % x.replace("_", "-"), required=False + ) sub_parser_action.add_argument("--page", required=False) sub_parser_action.add_argument("--per-page", required=False) |