fix: use lodash
Description
The repository use lodash.xx packages which is discouraged. Please consider reading offical lodash recommendations: https://lodash.com/per-method-packages
Motivation and Context
I just want to help on the maintainance here so I have removed unmaintained packages and replace them by the latest lodash release.
During the work available lint and tests forced me to update two more files:
- add a missing dependency on
@commitlint/config-nx-scopes - fix a lint error on
@commitlint/cli/src/cli.test.ts(test pass in both cases)
Usage examples
How Has This Been Tested?
I have run all existing script to try
Types of changes
- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
- [ ] My change requires a change to the documentation.
- [ ] I have updated the documentation accordingly.
- [ ] I have added tests to cover my changes.
- [x] All new and existing tests passed.
Thanks @jmfrancois ! Can you take care of the failing checks? Happy to merge this.
Sure will do that next week
We're using pure ESM now, so we should migrate to lodash-es instead.
We're using pure ESM now, so we should migrate to
lodash-esinstead.
@jmfrancois Would you mind rebasing? And wdyt about using lodash-es?
@jmfrancois ping?
This pull request is automatically built and testable in CodeSandbox.
To see build info of the built libraries, click here or the icon next to each commit SHA.
I will not have the time sorry to investigate why so many tests are failing. I prefer close the PR for now