-
-
Notifications
You must be signed in to change notification settings - Fork 2.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
Feature/#11 windows path issue #48
Feature/#11 windows path issue #48
Conversation
We'll need to do this in many more places. The tests are all using posix paths |
@@ -3,7 +3,7 @@ const path = require('path'); | |||
|
|||
const existsCache = new Map; | |||
|
|||
async function resolve(filepath, filenames, root = '/') { | |||
async function resolve(filepath, filenames, root = path.parse(filepath).root) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
or use root = path.delimiter
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On windows you don't get this \
or \\
or /
it depends on your volume C:\\
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! Will you also fix the tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests are hard to fix because I want to know whether it works on linux. I will wait for #35
Thanks, I'll get this released shortly. Want to check other places that might need to change too, @StarpTech? |
Use exact version of @parcel/source-map of 2.0.0-alpha.4.9 Approved-by: Maia Teegarden
Merge v2 2021-06-28 Approved-by: Joey Slater
No description provided.