-
Notifications
You must be signed in to change notification settings - Fork 104
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 default path set to '.' #311
Conversation
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.
Ah, yeah I see what you mean. I think it would make more sense to just correctly set the options.path
value in the constructor.
Currently it uses the value of assets-webpack-plugin/index.js Line 39 in cdfb655
|
Oh shoot, yeah you're right I forgot about that. Your change looks good then other than you should change the references from |
index.js
Outdated
@@ -39,6 +38,7 @@ AssetsWebpackPlugin.prototype = { | |||
? (compiler.options.output.path || '.') | |||
: (self.options.path || '.') | |||
) | |||
this.writer = createQueuedWriter(createOutputWriter(this.options)) |
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.
this.writer = createQueuedWriter(createOutputWriter(this.options)) | |
self.writer = createQueuedWriter(createOutputWriter(self.options)) |
The documentation says the default path is '.', however, without this fix, by the time createOutputWriter was called, options.path was null and the attempt to create the output directory would fail.
Okay, I've updated to use |
@rosenfeld should be available in |
Thanks! |
The documentation says the default path is '.', however, without this fix,
by the time createOutputWriter was called, options.path was null and
the attempt to create the output directory would fail.