-
Notifications
You must be signed in to change notification settings - Fork 85
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
🧹 refactor: replace lodash
with lodash-es
#378
Conversation
modules/front-end/angular.json
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @defqd , Thank you for your PR.
Have you check the recommended way in this answer? https://stackoverflow.com/a/62589268
With this, we don't need to add allowedCommonJsDependencies
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @cosmos-explorer, I checked. The warning has disappeared.
lodash
with lodash-es
modules/front-end/package.json
Outdated
@@ -21,6 +21,7 @@ | |||
"@angular/platform-browser-dynamic": "^14.0.0", | |||
"@angular/router": "^14.0.0", | |||
"@antv/g2": "^4.2.7", | |||
"lodash-es": "^4.17.21", | |||
"lodash.differenceby": "^4.8.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can remove
- lodash.differenceby
- lodash.get
- lodash.intersectionby
- lodash.keyby
Then rerun npm i
LGTM! Thank you very much |
@all-contributors |
I've put up a pull request to add @defqd! 🎉 |
Imported specific methods individually instead of import the whole 'lodash' library