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

Use native equivalents of lodash #6471

Open
trivikr opened this issue Nov 17, 2024 · 2 comments
Open

Use native equivalents of lodash #6471

trivikr opened this issue Nov 17, 2024 · 2 comments
Labels
feature-request A request for a new feature

Comments

@trivikr
Copy link
Contributor

trivikr commented Nov 17, 2024

Describe the Feature

As per the source code, ten different utilities of lodash are being used

social-app/package.json

Lines 152 to 161 in 0b38c77

"lodash.chunk": "^4.2.0",
"lodash.debounce": "^4.0.8",
"lodash.isequal": "^4.5.0",
"lodash.omit": "^4.5.0",
"lodash.once": "^4.1.1",
"lodash.random": "^3.2.0",
"lodash.samplesize": "^4.2.0",
"lodash.set": "^4.3.2",
"lodash.shuffle": "^4.2.0",
"lodash.throttle": "^4.1.1",

Up to five of them can be replaced by native code

These replacements can be used to reduce the number of lodash dependencies.

Attachments

No response

Describe Alternatives

No response

Additional Context

No response

@trivikr trivikr added the feature-request A request for a new feature label Nov 17, 2024
@trivikr
Copy link
Contributor Author

trivikr commented Nov 17, 2024

I noticed that 5 of the 10 lodash dependencies weren't used.
Posted PR to remove them in #6472

@PaulACoroneos
Copy link

Left a comment on your PR but I think moving to lodash-es may be nice move as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request A request for a new feature
Projects
None yet
Development

No branches or pull requests

2 participants