-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Bug: Migrate away from lodash method packages #18160
Comments
Based on https://lodash.com/per-method-packages, it sounds like it would make sense to switch, would be interested to see a PR. |
We removed We could consider replacing |
👍 to leave this as is until ESLint v10. |
I agree with leaving the package until ESLint v10.
|
I prepared #18179 to remove this and another use of The remaining one is blocked on ESLint v10. |
Lodash method packages are deprecated and will not receive any updated with Lodash 5, meaning they will be a security risk in the future.
Ref: https://lodash.com/per-method-packages
Currently
lodash.merge
is a dependency:eslint/package.json
Line 94 in e37153f
So the best would be to migrate to import
merge
directly fromlodash
.What did you expect to happen?
Do not use
lodash.merge
but importmerge
fromlodash
.Link to Minimal Reproducible Example
eslint/package.json
Line 94 in e37153f
Participation
The text was updated successfully, but these errors were encountered: