make --grep case insensitive? #808
Comments
|
+1 from me |
|
I have this forked branch https://github.com/pawanspace/mocha. grep is case insensitive. Another branch is https://github.com/pawanspace/mocha/tree/mocha-grep-ignorecase-optional it has option --ncgrep for case insensitive grep. I have not done so much of testing but basic functionality seems to be working. Let me know if it's not. thanks |
|
Hey, Since grep is now regexp based, I tried sending Thanks :) |
boneskull
added
type: feature
Add the ability to pass grep as a regexp-like string(query in the browser, and flag in the console). This improvement gives you pass a regexp that including a flag. e.g: '/[^0-9]/g', '/a/i', etc...
Add the ability to pass grep as a regexp-like string. (query in the browser or flag in the cli). This improvement gives you pass a regexp that including a flag, and solved mochajs#808 as well.
Add the ability to pass grep as a regexp-like string. (query in the browser or flag in the cli). This improvement gives you pass a regexp that including a flag, and solved mochajs#808 as well.
Add the ability to pass grep as a regexp-like string. (query in the browser or flag in the cli). This improvement gives you pass a regexp that including a flag, and solved mochajs#808 as well.
Add the ability to pass grep as a regexp-like string. (query in the browser or flag in the cli). This improvement gives you pass a regexp that including a flag, and solved mochajs#808 as well.
feature(mocha/grep): improve grep - issue #808
Add the ability to pass grep as a regexp-like string. (query in the browser or flag in the cli). This improvement gives you pass a regexp that including a flag, and solved #808 as well.
Add the ability to pass grep as a regexp-like string. (query in the browser or flag in the cli). This improvement gives you pass a regexp that including a flag, and solved #808 as well.
Add the ability to pass grep as a regexp-like string. (query in the browser or flag in the cli). This improvement gives you pass a regexp that including a flag, and solved #808 as well.
|
Closed via 6126e38 |
Add the ability to pass grep as a regexp-like string. (query in the browser or flag in the cli). This improvement gives you pass a regexp that including a flag, and solved #808 as well.
boneskull
removed
the
status: accepting prs
|
What was the conclusion on this issue? Case sensitivity plagues me daily - I miss tests when I'm greping because of case miss-match. Would a patch be accepted that adds a case-insensitive option for web-mocha and command-line mocha? --grep-ignore-case perhaps? |
|
@shanebdavis |
|
I would vote to change the default grep to be case-insensitive, when the grepped string is not "regex-like". This could be done in the next major version. I think the vast majority would find case-insensitive more valuable, and for the edge-cases where that is not desired, they can specify a regex to get exactly what they want. |
|
Is |
|
( |
No description provided.
The text was updated successfully, but these errors were encountered: