@-mention matching should be case-insensitive
Trott opened this issue · comments
btw:
[a-z0-9/-]
why is there noA-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)
We'll want to keep the test that was added, but yes, the code in index.js
can be reverted.