refactor: make arguments keyword only

Author: benediktziegler

commitizen/commands/check.py

@@ -76,8 +76,8 @@ def __call__(self):
             for commit in commits
             if not (
                 check := self.cz.validate_commit_message(
-                    commit.message,
-                    pattern,
+                    commit_msg=commit.message,
+                    pattern=pattern,
                     allow_abort=self.allow_abort,
                     allowed_prefixes=self.allowed_prefixes,
                     max_msg_length=self.max_msg_length,

commitizen/cz/base.py

@@ -98,6 +98,7 @@ def schema_pattern(self) -> str | None:
 
     def validate_commit_message(
         self,
+        *,
         commit_msg: str,
         pattern: str | None,
         allow_abort: bool,

docs/customization.md

@@ -323,6 +323,7 @@ from commitizen import git
 class CustomValidationCz(BaseCommitizen):
     def validate_commit_message(
         self,
+        *,
         commit_msg: str,
         pattern: str | None,
         allow_abort: bool,

tests/conftest.py

@@ -249,6 +249,7 @@ def schema_pattern(self):
 
     def validate_commit_message(
         self,
+        *,
         commit_msg: str,
         pattern: str | None,
         allow_abort: bool,

tests/test_cz_base.py

@@ -45,7 +45,13 @@ def test_schema(config):
 
 def test_validate_commit_message(config):
     cz = DummyCz(config)
-    assert cz.validate_commit_message("test", None, False, [], 0) == (True, [])
+    assert cz.validate_commit_message(
+        commit_msg="test",
+        pattern=None,
+        allow_abort=False,
+        allowed_prefixes=[],
+        max_msg_length=0,
+    ) == (True, [])
 
 
 def test_info(config):