feat: utility for showing system information

command to show system info output for bug reports

Issue #426

Author: sarsharma

.github/ISSUE_TEMPLATE/bug_report.md

@@ -28,11 +28,18 @@ If applicable, add screenshots to help explain your problem.
 
 ## Environment
 <!--
+For older commitizen versions, please include the
+output of the following commands manually:
+
 cz version
 python --version
 python3 -c "import platform; print(platform.system())"
 -->
-
+Add output of the following command to include the following
 - commitizen version:
 - python version:
 - operating system:
+```bash
+cz version --report
+```
+

CHANGELOG.md

@@ -1,4 +1,10 @@
 
+## v2.18.2 (2021-09-27)
+
+### Fix
+
+- **cli**: handle argparse different behavior after python 3.9
+
 ## v2.18.1 (2021-09-12)
 
 ### Fix

commitizen/__version__.py

@@ -1 +1 @@
-__version__ = "2.18.1"
+__version__ = "2.18.2"

commitizen/cli.py

@@ -227,6 +227,12 @@
                 ),
                 "func": commands.Version,
                 "arguments": [
+                    {
+                        "name": ["-r", "--report"],
+                        "help": "get system information for reporting bugs",
+                        "action": "store_true",
+                        "exclusive_group": "group1",
+                    },
                     {
                         "name": ["-p", "--project"],
                         "help": "get the version of the current project",
@@ -286,8 +292,13 @@ def main():
     # This is for the command required constraint in 2.0
     try:
         args = parser.parse_args()
-    except TypeError:
-        raise NoCommandFoundError()
+    except (TypeError, SystemExit) as e:
+        # https://github.com/commitizen-tools/commitizen/issues/429
+        # argparse raises TypeError when non exist command is provided on Python < 3.9
+        # but raise SystemExit with exit code == 2 on Python 3.9
+        if isinstance(e, TypeError) or (isinstance(e, SystemExit) and e.code == 2):
+            raise NoCommandFoundError()
+        raise e
 
     if args.name:
         conf.update({"name": args.name})

commitizen/commands/version.py

@@ -1,3 +1,6 @@
+import platform
+import sys
+
 from commitizen import out
 from commitizen.__version__ import __version__
 from commitizen.config import BaseConfig
@@ -9,9 +12,15 @@ class Version:
     def __init__(self, config: BaseConfig, *args):
         self.config: BaseConfig = config
         self.parameter = args[0]
+        self.operating_system = platform.system()
+        self.python_version = sys.version
 
     def __call__(self):
-        if self.parameter.get("project"):
+        if self.parameter.get("report"):
+            out.write(f"Commitizen Version: {__version__}")
+            out.write(f"Python Version: {self.python_version}")
+            out.write(f"Operating System: {self.operating_system}")
+        elif self.parameter.get("project"):
             version = self.config.settings["version"]
             if version:
                 out.write(f"{version}")

pyproject.toml

@@ -1,5 +1,5 @@
 [tool.commitizen]
-version = "2.18.1"
+version = "2.18.2"
 tag_format = "v$version"
 version_files = [
   "pyproject.toml:version",
@@ -29,7 +29,7 @@ exclude = '''
 
 [tool.poetry]
 name = "commitizen"
-version = "2.18.1"
+version = "2.18.2"
 description = "Python commitizen client tool"
 authors = ["Santiago Fraire <[email protected]>"]
 license = "MIT"

tests/commands/test_version_command.py

@@ -1,42 +1,72 @@
+import platform
+import sys
+
 from commitizen import commands
 from commitizen.__version__ import __version__
 
 
 def test_version_for_showing_project_version(config, capsys):
     # No version exist
-    commands.Version(config, {"project": True, "commitizen": False, "verbose": False})()
+    commands.Version(
+        config,
+        {"report": False, "project": True, "commitizen": False, "verbose": False},
+    )()
     captured = capsys.readouterr()
     assert "No project information in this project." in captured.err
 
     config.settings["version"] = "v0.0.1"
-    commands.Version(config, {"project": True, "commitizen": False, "verbose": False})()
+    commands.Version(
+        config,
+        {"report": False, "project": True, "commitizen": False, "verbose": False},
+    )()
     captured = capsys.readouterr()
     assert "v0.0.1" in captured.out
 
 
 def test_version_for_showing_commitizen_version(config, capsys):
-    commands.Version(config, {"project": False, "commitizen": True, "verbose": False})()
+    commands.Version(
+        config,
+        {"report": False, "project": False, "commitizen": True, "verbose": False},
+    )()
     captured = capsys.readouterr()
     assert f"{__version__}" in captured.out
 
     # default showing commitizen version
     commands.Version(
-        config, {"project": False, "commitizen": False, "verbose": False}
+        config,
+        {"report": False, "project": False, "commitizen": False, "verbose": False},
     )()
     captured = capsys.readouterr()
     assert f"{__version__}" in captured.out
 
 
 def test_version_for_showing_both_versions(config, capsys):
-    commands.Version(config, {"project": False, "commitizen": False, "verbose": True})()
+    commands.Version(
+        config,
+        {"report": False, "project": False, "commitizen": False, "verbose": True},
+    )()
     captured = capsys.readouterr()
     assert f"Installed Commitizen Version: {__version__}" in captured.out
     assert "No project information in this project." in captured.err
 
     config.settings["version"] = "v0.0.1"
-    commands.Version(config, {"project": False, "commitizen": False, "verbose": True})()
+    commands.Version(
+        config,
+        {"report": False, "project": False, "commitizen": False, "verbose": True},
+    )()
     captured = capsys.readouterr()
     expected_out = (
         f"Installed Commitizen Version: {__version__}\n" f"Project Version: v0.0.1"
     )
     assert expected_out in captured.out
+
+
+def test_version_for_showing_commitizen_system_info(config, capsys):
+    commands.Version(
+        config,
+        {"report": True, "project": False, "commitizen": False, "verbose": False},
+    )()
+    captured = capsys.readouterr()
+    assert f"Commitizen Version: {__version__}" in captured.out
+    assert f"Python Version: {sys.version}" in captured.out
+    assert f"Operating System: {platform.system()}" in captured.out