Skip to content
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

替换 lodash/merge 和 lodash/assignWith 为 deepmerge #6525

Closed
wants to merge 2 commits into from

Conversation

guoyunhe
Copy link
Contributor

@guoyunhe guoyunhe commented Jan 13, 2024

打包体积比较(越小越好)

lodash/merge+assignWith: Bundle size is 16.8 kB -> 6.68 kB (gzip)
              deepmerge: Bundle size is 2.18 kB -> 961 B (gzip)

数据来源 https://bundlejs.com/

@guoyunhe guoyunhe changed the title 替换 lodash/merge 为 deepmerge 替换 lodash/merge 和 lodash/assignWith 为 deepmerge Jan 13, 2024
Copy link
Contributor

PR preview has been successfully built and deployed to https://antd-mobile-preview-pr-6525.surge.sh

Copy link
Contributor

Deploy PR preview failed.

@zombieJ
Copy link
Member

zombieJ commented Feb 5, 2024

感谢 PR,从 CI 看这两个似乎不等价。我这边就先关啦,有需要可以开新的 PR 哈~

@zombieJ zombieJ closed this Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants