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

perf: remove _.get and _.set #54

Closed
wants to merge 2 commits into from
Closed

perf: remove _.get and _.set #54

wants to merge 2 commits into from

Conversation

noomorph
Copy link
Collaborator

@noomorph noomorph commented Oct 12, 2023

Resolves #42.

Replaces lodash.get and lodash.set dependencies with vanilla code.

Also, resolves an unlikely vulnerability which still reduces our Snyk Advisor score, so why not?

@noomorph noomorph force-pushed the build/optimize-deps branch from 8d58841 to 5213ff9 Compare October 27, 2023 09:11
@noomorph noomorph force-pushed the build/optimize-deps branch from 5213ff9 to dcff4c5 Compare October 27, 2023 09:31
@noomorph noomorph marked this pull request as ready for review October 27, 2023 09:39
@noomorph noomorph changed the title perf: reduce footprint perf: remove _.get and _.set Oct 27, 2023
@noomorph
Copy link
Collaborator Author

Merged in 4a55a4c.

@noomorph noomorph closed this Oct 27, 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.

Optimize lodash dependencies
1 participant