Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove lodash as runtime dependency #122

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Remove lodash as runtime dependency #122

wants to merge 1 commit into from

Conversation

navarroaxel
Copy link

No description provided.

@sshivananda sshivananda self-assigned this Feb 17, 2021
@crutchcorn
Copy link

@sshivananda is there any chance you'd be willing to look into this again?

Not only is this adding Lodash, but it appears that it's including all of lodash in a fairly non-treeshakable way. This adds 600KB of bundle size.

This might not be the worst problem on mobile apps, but I'm trying to use the package in React Native Web, which is a pretty substantial bundle increase.

@Monte9
Copy link
Owner

Monte9 commented Apr 2, 2024

I can take a look at this this week. Sorry about the delay!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants