Trott / remark-lint-prohibited-strings

remark-lint plugin to prohibit specified strings in markdown files

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

@-mention matching should be case-insensitive

Trott opened this issue · comments

btw: [a-z0-9/-] why is there no A-Z here?

I think that's a bug. It probably was intended to be case-insensitive. That part of the reg exp makes sure that we skip checking things that are part of an @-mention. So, even though Nodejs prohibits v8 as a string, it permits it in @nodejs/v8. This bug means that the skipping won't work for @Nodejs./v8 which would be a bug.

Originally posted by @Trott in #43 (comment)

i think this can be reverted with #41 case insensitive

We'll want to keep the test that was added, but yes, the code in index.js can be reverted.