-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix commitlint bug #186
Fix commitlint bug #186
Conversation
This simply adds the ability for the lexer to store the token type, as well as the value. Could prove handy, and really is standard procedure. Signed-off-by: Ole Petter <[email protected]>
Signed-off-by: Ole Petter <[email protected]>
Signed-off-by: Ole Petter <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Btw, I saw in Slack that someone had a problem with letter casing also (Changelog: title
instead of Changelog: Title
). It would be nice if we could accept either of those. I'm just thinking of contributors, it saves an entire round trip if they make a mistake, and rejecting lowercase is kind of silly. Do you agree?
advance() | ||
} | ||
# Require empty line afterwards in paragraph | ||
empty_line("A 'Changelog' over multiple lines needs to have a trailing empty line") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This won't barf if the changelog paragraph is the last line, will it?
Edit: Actually, I guess it will never happen, because signoff always comes after.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly :)
Yeh, I agree. Let's fix 💪 |
No description provided.