-
Notifications
You must be signed in to change notification settings - Fork 61
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
documentation: "keepEscaping" is missing #26
Comments
I've realized theres options.unescape but it doesn't seem to do anything in this case. |
That's not right, this is a bug, and it's inconsistent with how picomatch works. It should allow you to retain escaping. Thank you for reporting it. |
ah, wait. The option is Try this: console.log(braces("C:/Program Files \\(x86\\)", { keepEscaping: true })); |
Yes, that works. Then the documentation should be updated and it should be added to the documentation of micromatch. |
Agreed! Thanks for reporting this. Want to do a PR? |
I'm not sure if it's a braces or micromatch issue, but
returns
losing the escape characters, making
return the tokens
vs what I expected and what
returns
Here's some sample code: https://repl.it/repls/SharpStrangeChemistry
The text was updated successfully, but these errors were encountered: