refactor: replaces lodash - Githubissues

sverweij / dependency-cruiser

Validate and visualize dependencies. Your rules. JavaScript, TypeScript, CoffeeScript. ES6, CommonJS, AMD.

https://npmjs.com/dependency-cruiser

MIT License

5.27k stars 250 forks source link

refactor: replaces lodash #950

Closed sverweij closed 3 months ago

sverweij commented 3 months ago

Description

[x] centralizes remaining uses of lodash in central utilities
[x] in the central utilities replace the lodash array functions with native alternatives that work in our context
[x] in the central utilities replace the lodash object functions with native alternatives that work in our context
[x] removes lodash from the package manifest

Motivation and Context

lodash is a great library, however

it's not really maintained anymore
it has fairly large download size (1.4Mb according to pkg-size) - while we only use a small subset of it

How Has This Been Tested?

[x] green ci

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Documentation only change
[x] Refactor (non-breaking change which fixes an issue without changing functionality)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] :book:
- My change doesn't require a documentation update, or ...
- it does and I have updated it
[x] :balance_scale:
- The contribution will be subject to The MIT license, and I'm OK with that.
- The contribution is my own original work.
- I am ok with the stuff in CONTRIBUTING.md.

socket-security[bot] commented 3 months ago

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: npm/@types/lodash@4.17.7, npm/lodash@4.17.21

View full report↗︎

codeclimate[bot] commented 3 months ago

Code Climate has analyzed commit b4bf5577 and detected 0 issues on this pull request.

View more on Code Climate.