Skip to content
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(karma-webpack): Fix publicPath to be Windows-compatible #373

Merged
merged 3 commits into from
Nov 28, 2018

Conversation

matthieu-foucault
Copy link
Collaborator

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Taking over #366 (cleaned up PR and fixed another instance of publicPath). Fixes #362 (tested using steps provided by @johnnyreilly).

Public path is a URL, not a file path. It should not be created using path.join, which uses backslashes on Windows.

@jsf-clabot
Copy link

jsf-clabot commented Nov 27, 2018

CLA assistant check
All committers have signed the CLA.

@matthieu-foucault matthieu-foucault merged commit fca13b9 into master Nov 28, 2018
@matthieu-foucault matthieu-foucault deleted the fix-publicpath branch November 28, 2018 15:44
@johnnyreilly
Copy link

Great news!

@matthieu-foucault
Copy link
Collaborator Author

Published on npm under karma-webpack@next

@johnnyreilly
Copy link

johnnyreilly commented Nov 28, 2018

Awesome work - I'll give it a try!

TypeStrong/ts-loader#873

@johnnyreilly
Copy link

All good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dynamic import does not work on windows after 2.0.7 (re-open part deux)
4 participants