Conversation
TheLarkInn
left a comment
There was a problem hiding this comment.
We should probably include a deprecation message. I didn't see any in the diff. @bebraw what do you think?
|
@TheLarkInn 👍 for a deprecation message. That way people know to get rid of it. Always go with warnings in deprecations and then drop those in the next major version. |
|
But does 2 count as the next major version? Should it get a deprecation message in webpack 1? |
|
@Kovensky I guess the expectation is that people will try to run webpack 2 with the plugin enabled when they upgrade. So I would push the messages there and then drop in webpack 3. |
|
@bebraw since it outright conflicts with harmony modules, I guess it would be better to turn it into a noop that just prints the deprecation warning (also warning that it does nothing anymore)... |
I can do that... |
it's no longer needed for npm3 and causes many issues
196024a to
91cbb4c
Compare
|
As a kind of a replacement of the DedupePlugin, you can now use RootMostResolvePlugin from 'webpack-dependency-suite/plugins/root-most-resolve-plugin'. It's a See more info. |
|
or just |
|
@sokra |
we are also in this boat. our app is part of a lerna repo, so npm dedupe doesn't work well there. |
|
After some tests, I'm seeing multiple version of a few modules/files. Like with lodash. In a failry big app with a lot of dependencies, I can see What's the strategy now to dedupe these dependencies? I've tried |
|
I don't see any changes with this plugin. I still see multiple |
|
@tleunen Then it's quite possible that your package's dependencies have version-incompatible ranges with regards to lodash they want to use. I can add an option to But doing that can cause unexpected problems in case package A depends on lodash 2.x and package B on lodash 4.x, the method signatures might be incompatible. Also, did you add the plugin to the correct place in the config file (to |
|
@tleunen yeah, you should have two. So you still have 3? It would be best if I could implement an "error margin" function, so you could say, convert all |
DedupePlugin doesn't work correctly. It can cause this exception: "Uncaught TypeError: Cannot read property 'call' of undefined" - webpack/webpack#959 In a recent version of webpack, the plugin was changed to do nothing at all: webpack/webpack#3266

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
DedupePlugin is available but often doesn't work correctly.
It has problems with harmony modules.
What is the new behavior?
DedupePlugin is removed
Does this PR introduce a breaking change?
If this PR contains a breaking change, please describe the following...
Remove DedupePlugin when referenced directly