electron-userland/electron-json-storage

Replace lodash with lodash-es?

Nantris opened this issue · 4 comments

This appears to be the only reason that lodash is included in our bundles. It would be great to migrate to the more tree-shakeable lodash-es.

@jviotti pinging you on this because you seem to be the most active contributor.

A big thanks to you for all the great work!

Hey @slapbox ,

Sounds good to me! I haven't heard about lodash-es, but if you can send a quick PR, I'll issue a release ASAP. Thanks a lot!

@jviotti thanks for your reply!

I found out we actually have two other dependencies that depend on lodash, so this has moved down in priority for us quite a bit, but I'm still open to submitting a PR when I can.

PS thanks for your email - I saw it, but it's going to take me some time to get back to you since it's crunch time here right now.

No hurries. Sounds good!