Skip to content

Fixed Parser so that this Expression can work with ProvidePlugin#6588

Merged
sokra merged 1 commit intowebpack:masterfrom
nveenjain:fix/ParserProvidPlugin
Feb 27, 2018
Merged

Fixed Parser so that this Expression can work with ProvidePlugin#6588
sokra merged 1 commit intowebpack:masterfrom
nveenjain:fix/ParserProvidPlugin

Conversation

@nveenjain
Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?
BugFix

Did you add tests for your changes?
Yes

If relevant, link to documentation update:
N/A

Summary
Fixes #6526

Does this PR introduce a breaking change?
No

Other information
Now this expression have free property according to whether they are top level or not.

@webpack-bot
Copy link
Copy Markdown
Contributor

Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon.

@sokra sokra merged commit ec9f84e into webpack:master Feb 27, 2018
@sokra
Copy link
Copy Markdown
Member

sokra commented Feb 27, 2018

Thanks

@nveenjain nveenjain deleted the fix/ParserProvidPlugin branch February 27, 2018 08:12
@nveenjain
Copy link
Copy Markdown
Contributor Author

@sokra, i was having some doubts regarding this however, could you please answer #6526 (comment) . Thanks 😃

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.

this.jQuery cannot be shimmed with ProvidePlugin

3 participants