fix(cli): remove deprecated --all option in favor of --get-all

nejch · JohnVillalovos · commit e9d48cf69e0d · 2023-08-20T08:34:02.000-07:00
BREAKING CHANGE: The `--all` option is no longer available in the CLI. Use `--get-all` instead.
diff --git a/docs/faq.rst b/docs/faq.rst
@@ -42,7 +42,7 @@ Example::
 Not all items are returned from the API
 """""""""""""""""""""""""""""""""""""""
 
-If you've passed ``all=True`` (or ``--all`` via the CLI) to the API and still cannot see all items returned,
+If you've passed ``all=True`` to the API and still cannot see all items returned,
 use ``get_all=True`` (or ``--get-all`` via the CLI) instead. See :ref:`pagination` for more details.
 
 Common errors
diff --git a/gitlab/v4/cli.py b/gitlab/v4/cli.py
@@ -236,12 +236,6 @@ def _populate_sub_parser_by_class(
                 action="store_true",
                 help="Return all items from the server, without pagination.",
             )
-            sub_parser_action.add_argument(
-                "--all",
-                required=False,
-                action="store_true",
-                help="Deprecated. Use --get-all instead.",
-            )
 
         if action_name == "delete":
             if cls._id_attr is not None:
diff --git a/gitlab/v4/objects/commits.py b/gitlab/v4/objects/commits.py
@@ -155,6 +155,7 @@ class ProjectCommitManager(RetrieveMixin, CreateMixin, RESTManager):
         optional=("author_email", "author_name"),
     )
     _list_filters = (
+        "all",
         "ref_name",
         "since",
         "until",
diff --git a/tests/functional/cli/test_cli_repository.py b/tests/functional/cli/test_cli_repository.py
@@ -43,7 +43,15 @@ def test_list_all_commits(gitlab_cli, project):
     assert commit.id not in ret.stdout
 
     # Listing commits on other branches requires `all` parameter passed to the API
-    cmd = ["project-commit", "list", "--project-id", project.id, "--get-all", "--all"]
+    cmd = [
+        "project-commit",
+        "list",
+        "--project-id",
+        project.id,
+        "--get-all",
+        "--all",
+        "true",
+    ]
     ret_all = gitlab_cli(cmd)
     assert commit.id in ret_all.stdout
     assert len(ret_all.stdout) > len(ret.stdout)

Original file line number	Diff line number	Diff line change
`@@ -155,6 +155,7 @@ class ProjectCommitManager(RetrieveMixin, CreateMixin, RESTManager):`
`155`	`155`	`optional=("author_email", "author_name"),`
`156`	`156`	`)`
`157`	`157`	`_list_filters = (`
	`158`	`+ "all",`
`158`	`159`	`"ref_name",`
`159`	`160`	`"since",`
`160`	`161`	`"until",`