[Feature Request] Line length warning

[Lee-W]

Description

It's recommended to have only 72 characters. We could add a warning or a hint if the generated title might exceed 72.

Possible Solution

Add a line hint like vim

Additional context

Related Issue

[woile]

You mean for the commit subject? Where would we add this?

Remember that this is not a problem for the changelog. I'm not sure why the line length was chosen to be 72.

[Lee-W]

The first line of our generated commit. e.g., feat(new): awesome feature.
I'm thinking of make the functions in commitizen/utils.py and this funtionality something like commitizen-plugins. But it's still a rough idea.

I guess the number is from here

[woile]

IMO we should add it as a config parameter (max_subject_length), disabled by default, or set to 0 which would mean: anything.

I like the plugins idea, not sure how to hook everything haha but it's interesting

[Lee-W]

This could be my next target after #128 haha. Also, if my memory serves me right, you've mentioned implementing changelog-hooks (like email-hooks or slack-hooks). Do you have any idea how we should group them. Add them into to this repo as extra deps? Or, create another repo like commitizen-changelog-hooks?

[woile]

I was thinking more like separate packages, so people can choose which one to install:

pip install cz-slack-hook cz-email-hook

and then in your cz.toml

hooks = [
  "slack-hook",
  "email-hook"
] 

[jtpavlock]

I agree with the 72 character default for the commit title, as that's the max github displays in the commit history.

[Lee-W]

Here's a way to implement similar functionality #331 (comment). Not yet tried this functionality on cz-cli. might need some investigation

[changchaishi]

Hi there!
I found a feasible solution for line length warning. It is done by questionary's text validator, which will show a red color background warning at the bottom-left corner of the console.

the sample code is:

import questionary
from questionary import Validator, ValidationError

class NameValidator(Validator):
    def validate(self, document):
        text_len = len(document.text)
        if True:
            raise ValidationError(
                message=f"({text_len}/72)",
                cursor_position=text_len,
            )

questionary.text("Hi, type anything below:\n", validate=NameValidator).ask()

and result is:

This approach considered all text validation results are always False, so that it can keep up and update the warning text at below. Another approach is to stop the input cursor when text reaches maximum length.