-
Notifications
You must be signed in to change notification settings - Fork 73
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
feat: re-implement module concatenate #1295
Conversation
This reverts commit 3268dda.
Walkthrough这次改动涉及对模块合并的各个方面的调整,包括添加新模块、重构现有代码、简化逻辑和优化结构。有关模块连接、导出处理和优化的代码进行了明显的改进和更新。 Changes
Poem
Note 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 (
|
close #1231
improve #1165
inner 和 root transform 合并到 concatenate transform 中, concatenate transform 中 使用 var_link 的方式将 import/export 的符号 link 到原始的符号, 以保证 import/export reference 的性质。
转换成
转换成
Summary by CodeRabbit
新功能
module
模块中添加value
和setValue
导出项。index.js
引入了case
文件。文档
"index": "index.js"
修改为"index": "./index.js"
。e2e/fixtures/webpack.side-effects.empty-modules/expect.js
e2e/fixtures/javascript.tsx.preact/mako.config.json
mako.config.json
文件中,将optimization
配置中的concatenateModules
设置从false
更改为true
。