-
Notifications
You must be signed in to change notification settings - Fork 43
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
Goodbye lodash #100
Goodbye lodash #100
Conversation
28668b3
to
c7d1fb3
Compare
packages/ts/package.json
Outdated
@@ -61,6 +61,7 @@ | |||
"@types/lodash-es": "^4.17.6", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we remove this too?
We also need to remove the |
3993b5f
to
734787a
Compare
@reb-dev Thanks for pointing our the issues! I've just fixed along with a few other thing. |
734787a
to
eb21b20
Compare
eb21b20
to
ccc583d
Compare
This PR replaces lodash with our own functions and uses throttle from the
throttle-debounce
package, saving the users about 90KB in the final app bundle.Before
After