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

Incorrectly reporting "todo" item #5

Open
amimas opened this issue Jun 10, 2019 · 1 comment
Open

Incorrectly reporting "todo" item #5

amimas opened this issue Jun 10, 2019 · 1 comment
Labels

Comments

@amimas
Copy link

amimas commented Jun 10, 2019

I have a list in my markdown and one of the list item is like this:

- A `rel` or `path` contains non-supported characters.

    This includes spaces, control characters, and unsafe URI characters, For example: ```$ & + , / : ; = ? @ < > # % { } | \ ^~ [ ] ` ' "``` returns an error.

The above list item gets reported as containing a TODO. It seems the rule is catching this because there's a [ ] in the description paragraph of that list item.

The rule should be looking for [ ] at the beginning of a list item in order for it to be really a TODO item.

@azu azu added the bug label Jun 15, 2019
@azu
Copy link
Member

azu commented Jun 15, 2019

const match = text.match(/\[\s+\]\s/i);

This is caused by the above loose matching.

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

No branches or pull requests

2 participants