Feature: config-angular header-max-length should be 100

[FabienDehopre]

Expected Behavior

In the angular config package, the rule header-max-length should be set to 100 characters.

Any line of the commit message cannot be longer 100 characters!
cfr Angular Contributing Doc

Current Behavior

For the moment, the header line is configurered to only allow 72 characters.

Affected packages

• cli
• core
• prompt
• config-angular

Possible Solution

Configure the rule header-max-length to 100 characters.

Your Environment

Executable	Version
commitlint --version	6.1.0
git --version	git version 2.15.1.windows.2
node --version	v8.9.1

[FabienDehopre]

BTW, new rules should be created to restrict body and footer line length. For the moment, there are 2 rules to restrict both the body and footer length but not the line length used to create the body and footer.

These rules could be body-max-line-length and footer-max-line-length. What do you think about this?

[marionebl]

Thanks for the suggestions. To my knowledge the change from 72 to 100 char length in headers happened during the Angular.js to Angular transition, also see https://github.com/marionebl/commitlint/issues/146.

config-angular still tracks Angular.js, while there is no config that tracks Angular - they change their conventions to often to be useful outside of their project nowadays.

Concerning body-max-line-length, footer-max-line-length: Sure a PR implementing them would be welcome! Related issue: https://github.com/marionebl/commitlint/issues/208

[Fetz]

@marionebl since both body-max-line-length, footer-max-line-length was added to the rules should they be added to the config-angular? If yes what should be their value?

if 100 to be aligned to Angular, then we should change the header as well to 100
if not we should use 72 to be the same as the header

[marionebl]

@Fetz Let's go with 100 in config-conventional

[mightyiam]

I requested for 100 in in config-conventional, as well. #859 .

[escapedcat]

We released v9.0.1 under the next tag. This should include the updated max-length rule.
Give it a try and provide feedback. Thanks.