-
Notifications
You must be signed in to change notification settings - Fork 627
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 user config #1253
Feat user config #1253
Conversation
Orange-C
commented
Aug 14, 2019
- all user config handled together
- add devServer config
const { setBabelAlias } = require('rax-compile-config'); | ||
const UglifyJSPlugin = require('uglifyjs-webpack-plugin'); | ||
const OptimizeCSSAssetsPlugin = require('optimize-css-assets-webpack-plugin'); | ||
const { getBabelConfig, setBabelAlias } = require('rax-compile-config'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
webpack 的 config 是不是也可以提出去?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这份目前看只有app使用,如果未来其他plugin或者component那些都有了可以提
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SSR 的会复用这份 plugin 吧
@@ -108,8 +33,5 @@ module.exports = (context) => { | |||
render: serverRender.renderToString, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
为什么要从外面传一个 render 进去,而不是直接包掉这个逻辑
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
考虑到未来ice会使用react作为core的场景
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
不如直接传 renderToString 方法
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
不如直接传 renderToString 方法
字段名改成renderToString?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
看了下已经内置了rax的createElement逻辑了,还是把这个逻辑内置了吧,我改一下