Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add context option (options.context) #149

Merged
merged 1 commit into from
Oct 19, 2017

Conversation

cletusw
Copy link
Contributor

@cletusw cletusw commented Oct 5, 2017

Overrides webpack context, overridden by pattern context

Fixes #148

@michael-ciniawsky michael-ciniawsky changed the title feat: Plugin context option feat: add plugin context option (options.context) Oct 5, 2017
@@ -46,6 +47,15 @@ function CopyWebpackPlugin(patterns = [], options = {}) {
let contextDependencies;
const written = {};

let context;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

\n between the variable declaration and the if block please :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should I place it up with the other lets and consts as well?

@michael-ciniawsky michael-ciniawsky added this to the 4.2.0 milestone Oct 5, 2017
@michael-ciniawsky michael-ciniawsky requested review from alexander-akait and removed request for boopathi October 19, 2017 12:44
@michael-ciniawsky michael-ciniawsky changed the title feat: add plugin context option (options.context) feat: add context option (options.context) Oct 19, 2017
@michael-ciniawsky
Copy link
Member

@cletusw Please rebase when time :)

Overrides webpack context, overridden by pattern context

Fixes webpack-contrib#148
@cletusw
Copy link
Contributor Author

cletusw commented Oct 19, 2017

@michael-ciniawsky Rebased.

@cletusw
Copy link
Contributor Author

cletusw commented Oct 19, 2017

@michael-ciniawsky Awesome! Let me know when these patches get a version bump, thanks!

@michael-ciniawsky michael-ciniawsky removed this from the 4.3.0 milestone Nov 22, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants