-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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: support webpack 5 #6172
feat: support webpack 5 #6172
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6172 +/- ##
==========================================
- Coverage 85.71% 85.20% -0.51%
==========================================
Files 151 152 +1
Lines 3394 3427 +33
Branches 912 923 +11
==========================================
+ Hits 2909 2920 +11
- Misses 475 497 +22
Partials 10 10
Continue to review full report at Codecov.
|
@sorrycc 大佬 use @umijs/deps预编译后,webpack5能 |
@LeoHuiyi 这个文档是不是过时了,没有 OverridablesPlugin 这个文件了,其他 container 下的可以通过 webpack.container.xxx 引用到,参考 https://unpkg.com/browse/[email protected]/lib/index.js 和 https://unpkg.com/browse/[email protected]/lib/container/ |
然后如果有没有被 webpack 导出但是需要引用的文件,也可以提 FEATURE,我们往 webpack 上加就好了。 |
@sorrycc 明白,虽然这样 deps版本确定了,预编译 加载会更快。但是 很多包都改了源码,会有风险吗?基础库版本和社区同步会及时吗?感觉维护成本蛮大的。 |
Checklist
npm test
passesDescription of change