Closed ModyQyW closed 2 months ago
The core
package has been updated to replace the dependency lodash-unified
with lodash.groupby
. Accompanying this, a new dev dependency @types/lodash.groupby
has been added to ensure proper TypeScript support. Consequently, import statements in the source code have been adjusted to reflect this change.
Files | Change Summary |
---|---|
packages/core/package.json |
Updated dependency lodash-unified to lodash.groupby . Added @types/lodash.groupby as a new dev dependency. |
packages/core/src/utils.ts |
Changed import for groupBy from lodash-unified to lodash.groupby . |
In the codebase where changes flow,
A shift fromunified
togroupBy
did grow.
Dependencies align, types follow suit,
Our utils now play a harmonious flute.
🎶✨ A new chapter begins, with clear skies in view,
Streamlined and polished, for me and for you. 🐇💻
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Sorry, I changed it after seeing that element-plus
recommended lodash-unified
. I didn't know yarn had this error😱.
Description 描述
Import
groupBy
fromlodash.groupBy
instead oflodash-unified
. This action improves compatibility as yarn v1 doesn't install peerDependencies listed bylodash-unified
by default and may lead to errors.If we want to stay with
lodash-unified
, it is better to installlodash
,lodash-es
and@types/lodash-es
as dependencies. 🤔Linked Issues 关联的 Issues
Additional context 额外上下文
Summary by CodeRabbit
Chores
lodash-unified
tolodash.groupby
.@types/lodash.groupby
.Refactor
groupBy
utility function to improve modularity.