rorygarand / react-amplitude

React Amplitude Analytics
11 stars 13 forks source link

Reducing the end npm bundle size by switching to use the single lodas… #8

Closed boxman0617 closed 6 years ago

boxman0617 commented 6 years ago

…h modules

that are being used instead of the entire lodash library.

screen shot 2018-02-02 at 2 53 21 pm

^ that is the entire lodash lib being included within this lib, by allowing this PR, we can reduce that by quite a lot as you only use 5 functions from lodash.

On another note, would love to become a core contributor to this library! I think you have created an awesome package and want to help maintain it and make it better. Let me know what you think!

shawnphoffman commented 6 years ago

Bump. It would be really nice to get these changes merged in and pushed to the NPM package.

rorygarand commented 6 years ago

Yikes! I forgot about this... Just a sec.

rorygarand commented 6 years ago

I'll update npm shortly.