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 subject-full-stop false positive when using ellipsis #3839

Conversation

Mersho
Copy link
Contributor

@Mersho Mersho commented Jan 4, 2024

No description provided.

The SubjectFullStop failing test on
a title that ends with ellipsis.
@escapedcat
Copy link
Member

Happy to merge. In what situations do you use ... at the end of a commit subject?

@escapedcat escapedcat merged commit b4246d6 into conventional-changelog:master Jan 4, 2024
4 checks passed
@knocte
Copy link
Contributor

knocte commented Jan 4, 2024

Last time I was hit by this, I was writing something like "someType: fix typo sofisticated->sophisticated", but then my title was too long, so I wrote "someType: fix typo sofisticated->sophis..." and then commitlint yelled at me.

@escapedcat
Copy link
Member

https://github.com/conventional-changelog/commitlint/releases/tag/v18.4.4

@escapedcat
Copy link
Member

@knocte shouldn't you stop working by now? :P

knocte added a commit to knocte/commitlint that referenced this pull request Jan 20, 2024
Recent PR[1] didn't take in account very small subjects.

[1] conventional-changelog#3839
knocte added a commit to knocte/commitlint that referenced this pull request Jan 20, 2024
Recent PR[1] didn't take in account very small subjects.

[1] conventional-changelog#3839
knocte added a commit to knocte/commitlint that referenced this pull request Jan 20, 2024
Recent PR[1] didn't take in account very small subjects in
subject-full-stop rule.

[1] conventional-changelog#3839
knocte added a commit to knocte/commitlint that referenced this pull request Jan 21, 2024
Recent PR[1] didn't take in account very small subjects in
subject-full-stop rule.

[1] conventional-changelog#3839
knocte added a commit to knocte/commitlint that referenced this pull request Jan 21, 2024
Recent PR[1] didn't take in account very small subjects in
subject-full-stop rule.

[1] conventional-changelog#3839
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants