-
-
Notifications
You must be signed in to change notification settings - Fork 531
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
fix(plugin-webpack): correctly define the asset relocator base dir #2022
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2022 +/- ##
==========================================
+ Coverage 72.65% 72.74% +0.09%
==========================================
Files 74 74
Lines 2220 2220
Branches 420 420
==========================================
+ Hits 1613 1615 +2
- Misses 444 445 +1
+ Partials 163 160 -3
Continue to review full report at Codecov.
|
This seems reasonable, feel free to write tests for this. |
FYI the tests are failing on Windows due to differences in how paths are constructed. |
This commit should fix the tests on windows. However, I did not have a windows machine on me to test this.
Thanks for catching that! I pushed a commit that should fix it but I did not have a windows machine to verify. |
The tests expect the Windows paths' backslashes to be escaped. |
Am I correct to assume this is what Edit: Also, sorry for all this hassle. |
Seems so: > JSON.stringify('C:\foo')
'"C:\\foo"' |
Turns out JSON.stringify escapes backslashes
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.
Thanks for finishing it up!
Summarize your changes:
Closes #1801 which appears to have been abandoned.
Fixes #1559.