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.

Sign up for GitHub

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

Jump to bottom

fix: remove @antv/util and use lodash-es uniformly #495

Merged

pomelo-nwu merged 2 commits into master from dev/lodash

Jun 25, 2023

Contributor

Aarebecca commented Jun 25, 2023 •

edited

Loading

移除 @antv/util 包，统一使用 lodash-es
移除了内置 Util 方法 cloneDeep, uniqBy, deepEqual
修复构建脚本，默认开启 BundleAnalyzerPlugin 会导致后续子包构建阻塞。现在通过 npm run analyze 分析包体积
修复部分类型问题

@antv/util 占了接近 40K，包体积大小从 311K 缩小到 213 K，减小了 30% +

优化前

* 优化后

Aarebecca added 2 commits

June 25, 2023 22:20


          fix: remove @antv/util and use lodash-es uniformly

8091f60


          refactor: remove built-in uniqBy and deepEqual

7bdd586

Aarebecca force-pushed the dev/lodash branch from e45a3db to 7bdd586 Compare

June 25, 2023 14:21

Member

pomelo-nwu commented Jun 25, 2023

@Aarebecca good job！

pomelo-nwu merged commit 57acb46 into master

pomelo-nwu deleted the dev/lodash branch

June 25, 2023 14:28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet