-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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: ✨ 支持 external 为 function,函数接口和 webpack 对齐 #10180
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Size Change: +67 B (0%) Total Size: 9.46 MB
ℹ️ View Unchanged
|
Codecov ReportBase: 29.88% // Head: 29.87% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #10180 +/- ##
==========================================
- Coverage 29.88% 29.87% -0.01%
==========================================
Files 432 432
Lines 12409 12411 +2
Branches 2974 2974
==========================================
Hits 3708 3708
- Misses 8135 8137 +2
Partials 566 566
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
个人不想加,感觉 Object 格式的已够用,不够的场景推荐自行用 chainWebpack 。 |
就是因为通过webpackChain 加了 函数 external 配置,然后 mfsu 去调用这个函数。两边就都不符合预期了。 |
线下讨论,关闭 PR,不再框架层做特定场景的特殊处理。 |
但是 mfsu 调用 external 函数时 已经没有很多上下文信息,增加 __mfsu 来表明调用 来自 mfsu,做实现的兼容;
目前只能支持同步的 external。
ref: https://webpack.js.org/configuration/externals/#function