Skip to content
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

Merged
merged 3 commits into from
Apr 28, 2022
Merged

Conversation

oleorhagen
Copy link
Contributor

No description provided.

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]>
@oleorhagen oleorhagen requested a review from kacf April 28, 2022 07:29
Copy link
Member

@kacf kacf left a 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")
Copy link
Member

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exactly :)

@oleorhagen
Copy link
Contributor Author

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?

Yeh, I agree.

Let's fix 💪

@oleorhagen oleorhagen merged commit a9d8195 into mendersoftware:master Apr 28, 2022
@oleorhagen oleorhagen deleted the fix-commitlint-bug branch April 28, 2022 07:46
@oleorhagen
Copy link
Contributor Author

@kacf #187

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants