note that webpack 2 supports import()

[ysangkok]

No description provided.

[graingert]

probably worth deprecating this module now.

[ljharb]

@graingert absolutely not, it's still crucial for webpack 1 usage.

[graingert]

@ljharb but webpack 1 is old and broken now.

[ysangkok]

Is it not irrelevant whether one considers this repo obsolete or not? This pull request is just about notifying users that they don't need to use this with Webpack 2. Nothing more, nothing less.

[chicoxyzzy]

why do you think it's broken? most of people still use it for their projects

[ljharb]

@graingert actually webpack 2 is potentially much more broken, because it reifies more strongly incorrect ES module semantics that will definitely break when module loading in node is resolved, which is why Airbnb will likely not use webpack 2, and why I strongly recommend everybody avoid using non-babel import/export semantics (since Babel's broken semantics are popular enough that there will be a migration story).

[xzyfer]

it reifies more strongly incorrect ES module semantics

@ljharb can you please elaborate, or link to relevant discussion?

[ljharb]

@xzyfer I'm specifically referring to when you import a CJS module and use named imports (obviously when you import something that's written as an ES module in your own app, things are fine). Everyone's code that's doing import { PropTypes } from 'react' is going to break when node eventually ships modules.

[timdorr]

Since this was linked on reddit and before everyone throws @ljharb under the bus, first read this cogent comment explaining the situation. Jordan's just the messenger here. The actual spec is where this is all going down and where you should raise your pitchforks 😄

[FagnerMartinsBrack]

Does anybody have the link of any relevant discussion from Webpack's team about this?

Nevermind, it's right on Reddit, they opened a discussion about this (although as from the date of this comment it only has 2 comments).

[graingert]

@ysangkok can you rebase this PR?

[ysangkok]

@graingert ok now?