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

[Task] 优化 max 性能 #10645

Closed
5 tasks done
fz6m opened this issue Mar 1, 2023 Discussed in #10642 · 5 comments
Closed
5 tasks done

[Task] 优化 max 性能 #10645

fz6m opened this issue Mar 1, 2023 Discussed in #10642 · 5 comments

Comments

@fz6m
Copy link
Contributor

fz6m commented Mar 1, 2023

问题

经反馈,max g 运行需要 8s (#10642

环境

antd-pro + max (包含 umi-presets-pro

原因

和配置文件关系不大,主要在于模块预读取的瓶颈。

任务

Umi

优化前:

screenshot_2023-03-01_14-41-33

优化后:

screenshot_2023-03-01_14-27-34

openapi

需要 lazy import prettier 的导入,代码见 src/util.ts

- import * as prettier from 'prettier';
+ require('prettier') // 在使用时再导入

umi-presets-pro

alita

来源

Discussed in #10642

Originally posted by doornot March 1, 2023

@xiaohuoni
Copy link
Member

umi 新版本发布后 > alita 升级 umi 依赖版本 > alita 发版后 > umi-presets-pro 升级 alita 依赖发新版本,否则升级用户他们的底层 alita 版本不会变化。

用户直接重新安装,应该就会装到最新版本,因为 umi-presets-pro 里面没锁版本

alita 依赖的 umi 版本升级即可,建议不要锁

umi 发版本之后,需要确认内部的组件是否有影响,确认没影响之后,才会统一升 umi。所以依旧是锁的

@fz6m
Copy link
Contributor Author

fz6m commented Mar 10, 2023

用户直接重新安装,应该就会装到最新版本,因为 umi-presets-pro 里面没锁版本

由于间接依赖的大版本号仍然符合原来的设定,所以 pnpm 这种包管理器是不会提升仍符合 semver 范围的间接依赖版本的,故升级底层依赖一定要严格提升顶层依赖内的版本,这也是 changesets 要解的问题(会严格提升上游依赖内的版本号,保证用户升级时底层依赖也会升级),同时在 umi 里现在用了 workspace:* + pnpm publish 也可以彻底解这个问题了。

所以这次的性能提升,还是跟进完整个链路的版本提升,这样好。

umi 发版本之后,需要确认内部的组件是否有影响,确认没影响之后,才会统一升 umi。所以依旧是锁的

合理,没有意见。

@xiaohuoni
Copy link
Member

等待最后一步 umi-presets-pro 发版

@xiaohuoni
Copy link
Member

https://www.npmjs.com/package/umi-presets-pro/v/2.0.3 已发版

@fz6m
Copy link
Contributor Author

fz6m commented Mar 14, 2023

辛苦各位了 🌹

@fz6m fz6m closed this as completed Mar 14, 2023
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

No branches or pull requests

2 participants