refactor(cli): remove "--version" argument

BREAKING CHANGE: Use "cz verion" instead

Author: Lee-W

commitizen/cli.py

@@ -1,7 +1,6 @@
 import argparse
 import logging
 import sys
-import warnings
 from functools import partial
 
 from decli import cli
@@ -24,11 +23,6 @@
             "name": ["-n", "--name"],
             "help": "use the given commitizen (default: cz_conventional_commits)",
         },
-        {
-            "name": ["--version"],
-            "action": "store_true",
-            "help": "get the version of the installed commitizen",
-        },
     ],
     "subcommands": {
         "title": "commands",
@@ -277,16 +271,6 @@ def main():
     elif not args.name and not conf.path:
         conf.update({"name": "cz_conventional_commits"})
 
-    if args.version:
-        warnings.warn(
-            (
-                "'cz --version' will be deprecated in next major version. "
-                "Please use 'cz version' command from your scripts"
-            ),
-            category=DeprecationWarning,
-        )
-        args.func = commands.Version
-
     if args.debug:
         logging.getLogger("commitizen").setLevel(logging.DEBUG)
         sys.excepthook = commitizen_debug_excepthook

tests/test_cli.py

@@ -55,21 +55,6 @@ def test_ls(mocker, capsys):
     assert isinstance(out, str)
 
 
-def test_arg_version(mocker, capsys):
-    testargs = ["cz", "--version"]
-    mocker.patch.object(sys, "argv", testargs)
-
-    with pytest.warns(DeprecationWarning) as record:
-        cli.main()
-        out, _ = capsys.readouterr()
-        assert out.strip() == __version__
-
-    assert record[0].message.args[0] == (
-        "'cz --version' will be deprecated in next major version. "
-        "Please use 'cz version' command from your scripts"
-    )
-
-
 def test_arg_debug(mocker):
     testargs = ["cz", "--debug", "info"]
     mocker.patch.object(sys, "argv", testargs)