Skip to content

refactor: drop lodash #268

New issue

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

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

Already on GitHub? # to your account

Merged
merged 6 commits into from
Jun 26, 2025
Merged

refactor: drop lodash #268

merged 6 commits into from
Jun 26, 2025

Conversation

9romise
Copy link
Contributor

@9romise 9romise commented Jun 26, 2025

Resolves #267

At first, I was going to have lodash as devDependencies and bundle it in dist.
Finally, as mentioned in ota-meshi/yaml-eslint-parser#210, I chose to copy these methods into this repository from remeda and es-toolkit(has an installation size of 3.9 MiB).

All tests are passed on my machine:
image

Copy link
Member

@ota-meshi ota-meshi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's really awesome! Thank you so much!

@ota-meshi ota-meshi merged commit effe371 into vuejs:master Jun 26, 2025
8 checks passed
@9romise 9romise deleted the drop-lodash branch June 26, 2025 10:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider bundling lodash to reduce installation size
2 participants