-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[WEB-5027] fix: replace lodash package with lodash-es #7856
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
Conversation
Important Review skippedMore than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review. 91 files out of 198 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits. You can disable this status message by setting the Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. ✨ Finishing touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Pull Request Overview
This pull request replaces the lodash
package with lodash-es
across the entire monorepo to improve bundle performance through better tree-shaking support. The change enables ES modules to be imported individually, reducing final bundle sizes.
Changes:
- Replace all
lodash/module
imports withlodash-es/module
imports - Update package dependencies from
lodash
tolodash-es
in all package.json files - Convert some individual imports to barrel imports where multiple functions are used
Reviewed Changes
Copilot reviewed 198 out of 199 changed files in this pull request and generated 6 comments.
File | Description |
---|---|
pnpm-workspace.yaml | Update catalog dependencies from lodash to lodash-es |
packages/*/package.json | Replace lodash dependency with lodash-es in all packages |
Multiple .ts/.tsx files | Convert imports from lodash/module to lodash-es/module format |
apps/*/package.json | Update lodash dependencies to lodash-es in all applications |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
apps/web/core/components/workspace/sidebar/favorites/favorites.helpers.ts
Show resolved
Hide resolved
Linked to Plane Work Item(s) This comment was auto-generated by Plane |
Description
Type of Change