-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
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 :) |
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.
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: