textlint-rule / textlint-rule-en-capitalization

textlint rule that check capitalization in english text.
MIT License
4 stars 3 forks source link

Wrong warnings #7

Closed Mister-Hope closed 3 years ago

Mister-Hope commented 3 years ago

image

Besides, I don't think I should write 19Th instead of 19th.

image

I would like to consider first one as a bug(It don't contain english characters at all), and the second one as an enhance.

BTW: all of them is diplayed as can be autofixed, but none of them changes after I run with --fix

azu commented 3 years ago

Can you paste actual text as plain text?

Mister-Hope commented 3 years ago
- 19^th^

Which is parsed by markdown-it-sup, and will display as:

8-) :) :\* :( :-) :-( ;)

should be

😎 😃 😗 😦 😃 😦 😉

Mister-Hope commented 3 years ago

Infomation provided. Any progress on this issue?

azu commented 3 years ago

Thanks.

I confirmed.

When you use the number-started text, you get wrong result.

Example

1st line.
azu commented 3 years ago

Fixed in https://github.com/textlint-rule/textlint-rule-en-capitalization/releases/tag/v2.0.3