From bbb7df526f4375c438be97d8cfa0d9ea9d604e7d Mon Sep 17 00:00:00 2001 From: Thomas de Grenier de Latour Date: Tue, 25 Jan 2022 23:10:13 +0100 Subject: [PATCH 1/2] fix(cli): make 'timeout' type explicit --- gitlab/cli.py | 1 + 1 file changed, 1 insertion(+) diff --git a/gitlab/cli.py b/gitlab/cli.py index c4af4b8db..4bca0bfa5 100644 --- a/gitlab/cli.py +++ b/gitlab/cli.py @@ -181,6 +181,7 @@ def _get_base_parser(add_help: bool = True) -> argparse.ArgumentParser: "[env var: GITLAB_TIMEOUT]" ), required=False, + type=int, default=os.getenv("GITLAB_TIMEOUT"), ) parser.add_argument( From d493a5e8685018daa69c92e5942cbe763e5dac62 Mon Sep 17 00:00:00 2001 From: Thomas de Grenier de Latour Date: Tue, 25 Jan 2022 23:18:05 +0100 Subject: [PATCH 2/2] fix(cli): make 'per_page' and 'page' type explicit --- gitlab/cli.py | 1 + gitlab/v4/cli.py | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/gitlab/cli.py b/gitlab/cli.py index 4bca0bfa5..f06f49d94 100644 --- a/gitlab/cli.py +++ b/gitlab/cli.py @@ -197,6 +197,7 @@ def _get_base_parser(add_help: bool = True) -> argparse.ArgumentParser: "[env var: GITLAB_PER_PAGE]" ), required=False, + type=int, default=os.getenv("GITLAB_PER_PAGE"), ) parser.add_argument( diff --git a/gitlab/v4/cli.py b/gitlab/v4/cli.py index 7d8eab7f9..6830b0874 100644 --- a/gitlab/v4/cli.py +++ b/gitlab/v4/cli.py @@ -218,8 +218,8 @@ def _populate_sub_parser_by_class( f"--{x.replace('_', '-')}", required=False ) - sub_parser_action.add_argument("--page", required=False) - sub_parser_action.add_argument("--per-page", required=False) + sub_parser_action.add_argument("--page", required=False, type=int) + sub_parser_action.add_argument("--per-page", required=False, type=int) sub_parser_action.add_argument("--all", required=False, action="store_true") if action_name == "delete":