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

Ensure static and dynamic imports use the same binding #7479

Merged
merged 1 commit into from
Jun 5, 2018
Merged

Conversation

ooflorent
Copy link
Member

Add a test to ensure that #5153 has been fixed.
Fixes #5153

What kind of change does this PR introduce?

test

Did you add tests for your changes?

yes

Does this PR introduce a breaking change?

no

What needs to be documented once your changes are merged?

nothing

@webpack-bot
Copy link
Contributor

webpack-bot commented Jun 5, 2018

For maintainers only:

  • This need to be documented (issue in webpack/webpack.js.org will be filed when merged)

@webpack-bot
Copy link
Contributor

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

@sokra sokra merged commit b1a640e into master Jun 5, 2018
@sokra
Copy link
Member

sokra commented Jun 5, 2018

Thanks

@sokra sokra deleted the fix/5153 branch June 5, 2018 09:55
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 behavior is inconsistent with static import
3 participants