-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Update React & Webpack for webpack 2 #462
Comments
@DanielRosenwasser thoughts? |
@TheLarkInn thoughts? In my opinion, if ts-loader is Webpack 2-ready, ts-loader it is. |
It should be mentioned that code splitting with dynamic imports is currently unavailable microsoft/TypeScript#12364 |
just to clarify @chicoxyzzy, that's a deficiency of the typescript syntax right now. As a result, at-loader would be unable to handle this correct? also a large portion of why I wanted to move back to ts-loader was because it just appears to be better maintained.
|
It was just a side note. Both ts-loader and at-loader can't handle this because it is syntax error in TS. It should be clear that not all features of Webpack 2 are available with current version of TypeScript. This will prevent future misunderstanding. So if documentation will be updated for Webpack 2 it seems legit to add note about dynamic imports |
Great, thanks @mbertram |
Couple of things coming up:
Should we return this documentation to using ts-loader? for simple projects, the configuration is much easier, and the simplicity of that loader plugin means that its more likely to stay up to date long term.
Thoughts?
I'll start working on changes and a pull request sometime this weekend
DJF
The text was updated successfully, but these errors were encountered: