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

feat: remove lodash-es #648

Merged
merged 12 commits into from
Sep 14, 2023
Merged

feat: remove lodash-es #648

merged 12 commits into from
Sep 14, 2023

Conversation

Flowko
Copy link
Contributor

@Flowko Flowko commented Sep 10, 2023

resolves #642 and resolves #311

@vercel
Copy link

vercel bot commented Sep 10, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
ui ✅ Ready (Inspect) Visit Preview Sep 14, 2023 2:34pm

@Flowko Flowko changed the title feat: remove lodash feat: remove lodash-es Sep 10, 2023
@Flowko Flowko changed the title feat: remove lodash-es feat: remove lodash-es Sep 10, 2023
@danielroe
Copy link
Member

You are so fast.

@Flowko
Copy link
Contributor Author

Flowko commented Sep 10, 2023

I can close it, if you were planning to work on it, im suuuure you will do a better job than me, just let me know 🙏

@danielroe
Copy link
Member

No, let's do this! I would recommend also rewriting some of the code so it doesn't need these helper functions and is more obvious in location what it does. (Some of the more advanced use cases of lodash's helpers aren't actually used in the nuxt/ui code, I think.)

@benjamincanac benjamincanac merged commit d6476d1 into nuxt:dev Sep 14, 2023
1 check passed
Haythamasalama pushed a commit to Haythamasalama/ui that referenced this pull request Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

remove lodash Excessive lodash dependency loading
3 participants