Skip to content

Commit 2be6bcb

Browse files
committed
fix(check): honor message length limit from cli and config
CLI > config > default (0) for not limit
1 parent 092ec64 commit 2be6bcb

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

commitizen/commands/check.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,12 @@ def __init__(self, config: BaseConfig, arguments: CheckArgs, *args: object) -> N
4646
)
4747

4848
self.use_default_range = bool(arguments.get("use_default_range"))
49-
self.max_msg_length = arguments.get(
50-
"message_length_limit", config.settings.get("message_length_limit", 0)
49+
50+
message_length_limit = arguments.get("message_length_limit")
51+
self.message_length_limit: int = (
52+
message_length_limit
53+
if message_length_limit is not None
54+
else config.settings["message_length_limit"]
5155
)
5256

5357
# we need to distinguish between None and [], which is a valid value
@@ -100,7 +104,7 @@ def __call__(self) -> None:
100104
pattern=pattern,
101105
allow_abort=self.allow_abort,
102106
allowed_prefixes=self.allowed_prefixes,
103-
max_msg_length=self.max_msg_length,
107+
max_msg_length=self.message_length_limit,
104108
commit_hash=commit.rev,
105109
)
106110
).is_valid

0 commit comments

Comments
 (0)