Drop lodash from the package

webpack-contrib / webpack-bundle-analyzer

Webpack plugin and CLI utility that represents bundle content as convenient interactive zoomable treemap

MIT License

12.58k stars 488 forks source link

Drop lodash from the package #393

Closed TrySound closed 1 year ago

TrySound commented 4 years ago

Lodash is quite big package and often overused for things available natively. In this diff I removed it from package dependencies and left only for tests.

th0r commented 3 years ago

Thanks for the PR, but I don't want to drop lodash as JS std library is not good enough to replace it yet and code becomes more complex and less obvious.

TrySound commented 3 years ago

Well, I spent a lot of time digging through not very obvious lodash solutions.

TrySound commented 3 years ago

Also really in most cases it's just each -> forEach, filter -> filter, map -> map. Are these cases less obvious?

TrySound commented 3 years ago

Ok. Let's step away from code complication and I will do only drop in replacement.

onigoetz commented 2 years ago

Hi, I'm a bit late to the party, is this PR still in progress ?

I'm also trying to remove lodash from my dependencies and would help reduce the install size of my tools.