-
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(mf): 新增remoteHash参数控制mf的产物是否启用hash(#11711) #11714
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Co-authored-by: 咲奈Sakina <[email protected]>
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #11714 +/- ##
=======================================
Coverage 28.93% 28.94%
=======================================
Files 485 485
Lines 14771 14775 +4
Branches 3498 3501 +3
=======================================
+ Hits 4274 4276 +2
- Misses 9737 9739 +2
Partials 760 760
☔ View full report in Codecov by Sentry. |
@consistent-k PR 本身 没有问题, 没有了 hash 的风险在于,如果一次初始加载和异步加载之间有一次发布的话,构建产物不是同一批的可能会有问题。(比如 module id 和实际 module 的对应关系是不同的; treeshake 导致一个模块内的代码也是不同的) |
推荐的做法 还是有一个配置中心,下发一个当前使用的 remote 的 url,这样产物始终用都是同一次构建的内容。 |
这么做的原因就是因为我们还没有这个配置中心,但是又想让其他子项目快速的使用MF的功能,配置中心的建设可能需要先跑通MF给领导看了之后才能提下一步计划。 |
嗯 这个风险已知,我们刚开始在项目上尝试MF,可能需要一步一步来,已知的缓存问题我会按照你的建议和领导提建设配置中心的想法。 |
因为这个需求之前也有好几个人在 issue 中穿插提到过,对于小公司,小范围场景使用的话,一般只需要承担 所以我认为这个 PR 虽然有风险,但是更适合于相当一部分人的需求,是有意义的。 |
新增remoteHash参数来控制mf的产物是否启用hash