Commitlint: footer-leading-blank is complaining when there is no footer

Created on 1 Sep 2016  Â·  4Comments  Â·  Source: conventional-changelog/commitlint

When attempting to submit a commit:

chore: force add README.md
This is designed to be used in combination with .gitignore ./README.md when dynamically generating documentation.

I am getting:

â§— input: chore: force add README.md
✖ footer must have leading blank line [footer-leading-blank]
âš  tense of body must be present-imperative. Verbs in other tenses: is - present-third-person,generating - present-participle,starting - present-participle,is - present-third-person,commits - present-third-person,staged - > past-tense [body-tense]
âš  header must not be shorter than 30 characters [header-min-length]
âš  message must be lowerCase [subject-case]
✖ found 1 problems, 3 warnings

There is no footer, therefore there must be no warning.

bug

Most helpful comment

This issue is part of the roadmap to version 1 / a planned commitlint renamer. I'll pick it up as soon as possible, stay tuned.

Thanks for the additional detail!

All 4 comments

I'm running into this issue as well, the footer isn't a requirement, is it?

If it helps this error isn't thrown if you only have a subject and leave out the body and footer. However, when a body is included without a footer, this error seems to always get thrown.

This issue is part of the roadmap to version 1 / a planned commitlint renamer. I'll pick it up as soon as possible, stay tuned.

Thanks for the additional detail!

@marionebl no problem, thanks for the awesome tool! Glad to hear you're actively working on the project.

Side Note: Is there any way to ignore tense for certain patterns (e.g. with a regex or something)? If not, maybe I'll create another ticket with that as a feature request. For example, I have a package name that ends with -fetching and, when i mention in the commit subject, it throws a warning. (Arguably I should maybe rename that package, but I could definitely see this being useful in a variety of cases)

@skipjack A feature request would be cool

Was this page helpful?
0 / 5 - 0 ratings

Related issues

MathiasKandelborg picture MathiasKandelborg  Â·  3Comments

thomasthiebaud picture thomasthiebaud  Â·  5Comments

yenbekbay picture yenbekbay  Â·  5Comments

pixelass picture pixelass  Â·  3Comments

padamstx picture padamstx  Â·  4Comments