Yuriy Grunin

Results 76 comments of Yuriy Grunin

> Actually, lets say that webpack is doing the right thing. Technically the path is correct, so I can see them won't fixing that. @dzearing I'm not saying that webpack...

@dzearing So what do you think about filling an issue in webpack repo?

@constgen [this callback](https://github.com/webpack-contrib/webpack-bundle-analyzer/blob/d09366690257ca69b2b0639cf20f91c0a46f0c0c/src/BundleAnalyzerPlugin.js#L37) is called on every bundle recompilation. So, for `analyzerMode: 'server'` it calls [startAnalyzerServer](https://github.com/webpack-contrib/webpack-bundle-analyzer/blob/d09366690257ca69b2b0639cf20f91c0a46f0c0c/src/BundleAnalyzerPlugin.js#L98) method which calls [updateChartData](https://github.com/webpack-contrib/webpack-bundle-analyzer/blob/d09366690257ca69b2b0639cf20f91c0a46f0c0c/src/BundleAnalyzerPlugin.js#L100) method to push changes to the client via websocket.

I don't think it's possible. How it should work if module A imports something from module B and vice versa? It's a perfectly valid situation in ES modules.

I don't understand how it should work. For example, you have 100 components that import react and 10 other vendor libs. Do you want all of them to have react...

> Note that if `stats.json` isn't in the same directory as where webpack puts the bundles, the `webpack-bundle-analyzer` CLI won't find them. Another bug? In this case you need to...

Never heard of Brotli as well. The only concern I have is the increasing of analyze time. Agree with @valscion here - it would be nice to see a proof-of-concept...

@samccone as I understand you've made your own visualization tool, right?

Well, I had thoughts about it. Maybe we'll come back to it a little later after implementing a few other important features e.g. import reasons. P.S. This duplication logic can...