Skip to content

feat: 1. 支持chainWebpack配置 2. 添加externals配置减小物料体检 3. windows下路径优化#45

Closed
flgame wants to merge 0 commit intoalibaba:mainfrom
flgame:main
Closed

feat: 1. 支持chainWebpack配置 2. 添加externals配置减小物料体检 3. windows下路径优化#45
flgame wants to merge 0 commit intoalibaba:mainfrom
flgame:main

Conversation

@flgame
Copy link
Copy Markdown

@flgame flgame commented Mar 11, 2024

  1. 支持chainWebpack配置
  2. 添加externals配置减小物料体积
  3. windows下路径优化

以上修改已在内部项目应用并验证

@flgame
Copy link
Copy Markdown
Author

flgame commented Mar 21, 2024

@eternalsky 麻烦review下看看是否可行

@eternalsky
Copy link
Copy Markdown
Collaborator

@flgame 建议三个部分分开提 pr,这样部分合并的速度会快一些。
这里的三个 feature,

  1. 感觉没有必要,build-scripts 天生就提供 webpack-chain 的功能,不需要在某一个插件里再实现一遍。
  2. external 的标准无法判断,贸然加入其他的,对已经使用内置的用户来说是 break change,因此提交新的 external 之前应该先讨论其必要性,因为 external 本来也可以通过 1 的能力自己加入。
  3. 这个没有问题

@flgame
Copy link
Copy Markdown
Author

flgame commented May 14, 2024

@eternalsky 好的,我拆分下

  1. 没找到配置方式,可以提供下如何配置
  2. 原默认external配置里已有antd、next,只是并不完善,且当前提供的external配置并不支持function的方式,无法达到预期效果。目前除了echarts,其它应该都是低代码编辑器、运行时需要全局加载的,看看需要如何修改

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants