-
Notifications
You must be signed in to change notification settings - Fork 106
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
error on Windows #202
Comments
Can you post the full file / directory path (what's being chopped after the |
Also which version of Node? The fix for Windows path separators being treated as escape patterns has some issues in Node 8 and different issues in Node 11 that aren't in Node 10 |
I think the regexp introduced in #193 is not complete/broken on Node 10. cc @mafintosh |
I think this line Line 36 in 1cfbc56
needs to change the pattern from |
There's a bunch of issues with that regex that are fixed in the PR I'm working on, for instance it gets confused by unicode chars like //u12a3 - but in my tests I've only seen that happen on Node 8. What's the text after the Maybe it's mistakenly parsing something like D:/a/n as a newline? |
That happens on Azure pipelines. There is nothing else. |
was this ever resolved? |
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
This issue was closed because it has been stalled for 5 days with no activity. |
The text was updated successfully, but these errors were encountered: