Merge pull request #141 from Lee-W/batch-check

Batch check

Author: woile

commitizen/bump.py

@@ -6,7 +6,6 @@
 
 from packaging.version import Version
 
-from commitizen.git import GitCommit
 from commitizen.defaults import (
     MAJOR,
     MINOR,
@@ -15,6 +14,7 @@
     bump_message,
     bump_pattern,
 )
+from commitizen.git import GitCommit
 
 
 def find_increment(

commitizen/cli.py

@@ -151,13 +151,18 @@
                 "arguments": [
                     {
                         "name": "--commit-msg-file",
-                        "required": True,
                         "help": (
                             "ask for the name of the temporal file that contains "
                             "the commit message. "
                             "Using it in a git hook script: MSG_FILE=$1"
                         ),
-                    }
+                        "exclusive_group": "group1",
+                    },
+                    {
+                        "name": "--rev-range",
+                        "help": ("a reange of git rev to check. e.g, master..HEAD"),
+                        "exclusive_group": "group1",
+                    },
                 ],
             },
             {

commitizen/commands/__init__.py

@@ -3,11 +3,10 @@
 from .commit import Commit
 from .example import Example
 from .info import Info
+from .init import Init
 from .list_cz import ListCz
 from .schema import Schema
 from .version import Version
-from .init import Init
-
 
 __all__ = (
     "Bump",

commitizen/commands/bump.py

@@ -1,15 +1,15 @@
-from typing import Optional, List
+from typing import List, Optional
 
 import questionary
 from packaging.version import Version
 
 from commitizen import bump, factory, git, out
 from commitizen.config import BaseConfig
 from commitizen.error_codes import (
+    COMMIT_FAILED,
     NO_COMMITS_FOUND,
-    NO_VERSION_SPECIFIED,
     NO_PATTERN_MAP,
-    COMMIT_FAILED,
+    NO_VERSION_SPECIFIED,
     TAG_FAILED,
 )
 

commitizen/commands/check.py

@@ -1,15 +1,16 @@
 import os
 import re
+from typing import Dict
 
-from commitizen import factory, out
+from commitizen import factory, git, out
 from commitizen.config import BaseConfig
 from commitizen.error_codes import INVALID_COMMIT_MSG
 
 
 class Check:
     """Check if the current commit msg matches the commitizen format."""
 
-    def __init__(self, config: BaseConfig, arguments: dict, cwd=os.getcwd()):
+    def __init__(self, config: BaseConfig, arguments: Dict[str, str], cwd=os.getcwd()):
         """Init method.
 
         Parameters
@@ -21,31 +22,53 @@ def __init__(self, config: BaseConfig, arguments: dict, cwd=os.getcwd()):
             the flags provided by the user
 
         """
+        self.commit_msg_file: str = arguments.get("commit_msg_file")
+        self.rev_range: str = arguments.get("rev_range")
+
+        self._valid_command_argument()
+
         self.config: BaseConfig = config
         self.cz = factory.commiter_factory(self.config)
-        self.arguments: dict = arguments
+
+    def _valid_command_argument(self):
+        if bool(self.commit_msg_file) is bool(self.rev_range):
+            out.error("One and only one argument is required for check command!")
+            raise SystemExit()
 
     def __call__(self):
-        """Validate if a commit message follows the conventional pattern.
+        """Validate if commit messages follows the conventional pattern.
 
         Raises
         ------
         SystemExit
             if the commit provided not follows the conventional pattern
 
         """
-        commit_msg_content = self._get_commit_msg()
+        commit_msgs = self._get_commit_messages()
         pattern = self.cz.schema_pattern()
-        if self._has_proper_format(pattern, commit_msg_content) is not None:
-            out.success("Commit validation: successful!")
-        else:
-            out.error("commit validation: failed!")
-            out.error("please enter a commit message in the commitizen format.")
-            raise SystemExit(INVALID_COMMIT_MSG)
-
-    def _get_commit_msg(self):
-        temp_filename: str = self.arguments.get("commit_msg_file")
-        return open(temp_filename, "r").read()
-
-    def _has_proper_format(self, pattern, commit_msg):
+        for commit_msg in commit_msgs:
+            if not Check.validate_commit_message(commit_msg, pattern):
+                out.error(
+                    "commit validation: failed!\n"
+                    "please enter a commit message in the commitizen format.\n"
+                    f"commit: {commit_msg}\n"
+                    f"pattern: {pattern}"
+                )
+                raise SystemExit(INVALID_COMMIT_MSG)
+        out.success("Commit validation: successful!")
+
+    def _get_commit_messages(self):
+        # Get commit message from file (--commit-msg-file)
+        if self.commit_msg_file:
+            with open(self.commit_msg_file, "r") as commit_file:
+                commit_msg = commit_file.read()
+            return [commit_msg]
+
+        # Get commit messages from git log (--rev-range)
+        return [commit.message for commit in git.get_commits(end=self.rev_range)]
+
+    @staticmethod
+    def validate_commit_message(commit_msg: str, pattern: str) -> bool:
+        if commit_msg.startswith("Merge") or commit_msg.startswith("Revert"):
+            return True
         return re.match(pattern, commit_msg)

commitizen/commands/commit.py

@@ -5,14 +5,14 @@
 import questionary
 
 from commitizen import factory, git, out
-from commitizen.cz.exceptions import CzException
 from commitizen.config import BaseConfig
+from commitizen.cz.exceptions import CzException
 from commitizen.error_codes import (
-    NO_ANSWERS,
     COMMIT_ERROR,
+    CUSTOM_ERROR,
+    NO_ANSWERS,
     NO_COMMIT_BACKUP,
     NOTHING_TO_COMMIT,
-    CUSTOM_ERROR,
 )
 
 

commitizen/commands/init.py

@@ -1,12 +1,11 @@
-from packaging.version import Version
-
 import questionary
+from packaging.version import Version
 
 from commitizen import factory, out
+from commitizen.config import BaseConfig, IniConfig, TomlConfig
 from commitizen.cz import registry
-from commitizen.config import BaseConfig, TomlConfig, IniConfig
-from commitizen.git import get_latest_tag_name, get_tag_names
 from commitizen.defaults import long_term_support_config_files
+from commitizen.git import get_latest_tag_name, get_tag_names
 
 
 class Init:

commitizen/commands/list_cz.py

@@ -1,6 +1,6 @@
 from commitizen import out
-from commitizen.cz import registry
 from commitizen.config import BaseConfig
+from commitizen.cz import registry
 
 
 class ListCz:

commitizen/commands/version.py

@@ -1,6 +1,6 @@
 from commitizen import out
-from commitizen.config import BaseConfig
 from commitizen.__version__ import __version__
+from commitizen.config import BaseConfig
 
 
 class Version:

commitizen/config/__init__.py

@@ -4,9 +4,10 @@
 
 from commitizen import defaults, git, out
 from commitizen.error_codes import NOT_A_GIT_PROJECT
+
 from .base_config import BaseConfig
-from .toml_config import TomlConfig
 from .ini_config import IniConfig
+from .toml_config import TomlConfig
 
 
 def load_global_conf() -> Optional[IniConfig]: