-
Notifications
You must be signed in to change notification settings - Fork 85
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
chore: 🔧 adjust puffin for easy profile for both long time compiling and watch mode #1168
Conversation
Walkthrough这次更新主要涉及条件导入、未使用导入的删除、异步代码重构以用于性能分析,以及在闭包中传递不同的参数。 Changes
Poem
Note Pull Request Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://coderabbit.ai TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
…and watch mode (#1168) * chore: 🔧 adjust puffin for easy profile for long time compiling * refactor: 🎨 move profile logic to profile app
* feat: ✨ add [d]ynamic [r]equire runtime function * refactor: 🎨 dynamic import require process with interop * test: ✅ add test case * test: ✅ import context module adjust to dynamic import * release: @umijs/mako@0.4.18-canary.20240517.1 * chore: ⬆️ mako 0.4.18-canary.20240517.1 * chore: ⬆️ update pnpm-lock * typo * fix: fix "too many files open" error when watching with-antd example (#1022) * fix: fix "too many files open" error when watching with-antd example * fix: include dist dictionary of root's parent dictionary when watching --------- Co-authored-by: zp365238 <zp365238@antgroup.com> Co-authored-by: pshu <pishu.spf@antfin.com> * chore: 🔧 adjust puffin for easy profile for both long time compiling and watch mode (#1168) * chore: 🔧 adjust puffin for easy profile for long time compiling * refactor: 🎨 move profile logic to profile app * fix: decorator visitor should run before preset env (#1176) * test: ✅ add runtime assert in decorator and target to chrome 40 * fix: 🐛 move decorator before preset_env * add comment on decorators * feat: add watch.ignorePaths config (#1179) * release: @umijs/mako@0.4.18-canary.20240520.1 * chore: ⬆️ v0.4.18-canary.20240520.1 * chore: ⬆️ update pnpm-lock --------- Co-authored-by: zhangpanweb <37805064+zhangpanweb@users.noreply.github.com> Co-authored-by: zp365238 <zp365238@antgroup.com> Co-authored-by: chencheng (云谦) <sorrycc@gmail.com>
Summary by CodeRabbit
main.rs
中,根据特性进行条件导入,删除未使用的导入,重构异步代码以进行性能分析,更改闭包中的参数传递。utils/mod.rs
中,为profile_gui
模块的声明添加了条件编译属性#[cfg(feature = "profile")]
。profile_gui.rs
中,重构以使用Compiler
结构而不是Notify
,调整初始化逻辑,并根据编译器上下文参数进行编译和服务器生成。