-
Notifications
You must be signed in to change notification settings - Fork 39
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
add import loader (since webpack v2.1.0-beta.28) #90
add import loader (since webpack v2.1.0-beta.28) #90
Conversation
@brandonroberts Please help to review this PR since webpack 3 has removed |
docs/options.md
Outdated
|
||
If you prefer to use `dynamic import`, set the `loader` to `import` | ||
|
||
**NOTE:** Using `import` only works with Webpack >= 2.1.0-beta.28. |
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.
Change to Webpack => 2.1.0
since its already been officially released
docs/options.md
Outdated
} | ||
``` | ||
|
||
If you prefer to use `dynamic import`, set the `loader` to `import` |
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.
Remove If you prefer
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.
@brandonroberts I add If you prefer
here because it is also mentioned in If you prefer to use System.import, set the loader to system
, so should I remove that also?
@JounQin will you add a test to verify the |
Thanks! |
Published as |
https://github.com/webpack/webpack/releases/tag/v2.1.0-beta.28