Skip to content

Don't force internal requires of options #171

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

Merged
merged 1 commit into from
Feb 11, 2017
Merged

Conversation

jescalan
Copy link
Contributor

Hi there! So I ran into a situation where I need to require the parser myself and pass it in, rather than having it required internally, so I added this option. I couldn't quickly get my head around how the test suite works, but if you give me a push in the right direction I'd be happy to add a test for this. Meanwhile, it is a progressive upgrade, so it won't break any existing functionality.

@ai
Copy link
Contributor

ai commented Feb 10, 2017

Good change. I will accept it and release on this weekend (sorry, I am in the travel right now).

if options need to be required from a specific destination, or are
passed in by a different source, this is necessary
`
@ai ai merged commit 8c897ca into webpack-contrib:master Feb 11, 2017
@ai
Copy link
Contributor

ai commented Feb 11, 2017

Released in 1.3.0.

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

Successfully merging this pull request may close these issues.

2 participants