Skip to content
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

roadhog 2.0 发布 #55

Open
sorrycc opened this issue Jan 2, 2018 · 110 comments
Open

roadhog 2.0 发布 #55

sorrycc opened this issue Jan 2, 2018 · 110 comments
Labels

Comments

@sorrycc
Copy link
Owner

sorrycc commented Jan 2, 2018

趁着元旦,roadhog 2.0 发布了,我们一起来看看有啥变化吧。

👏 👏 👏
 

一、

首先,我们把底层重写,基于 af-webpack

为啥要基于 af-webpack?单看 roadhog,这意义不大,但如果有数个工具都基于 af-webpack 做上层封装,那么就意味着 webpack 工具层的配置统一。 尤其对我们内部来说,有 4-5 个工具,每个工具有各自的配置方法,对于开发者来说无疑是个灾难。对外部的同学来说,umi 和 roadhog 都基于 af-webpack,那么之后在这两个工具之间切换,webpack 的配置层是无需变动的。

另外,大家会发现 roadhog 的 issue 维护不太积极,是因为我的工作重心目前不在这,而是在 umi 上。所以切换到 af-webpack,af-webpack 是 umi 的底层共建,这样就可以变相地保证我在这个项目上的投入。

同时,为了更通用,配置文件从 .roadhogrc 更名为 .webpackrc

 

二、

然后,在体验上,保持了和 create-react-app 最新版一致。像是 redbox 显示出错信息、HMR、出错点击后跳转到 IDE、ESLint 出错提示等等。

并在此基础上,增加了:

  • 所有的配置都能自动生效(热更新或自重启)
  • dev server 断线重连
  • 配置项校验和提醒(同时出现在浏览器和控制台里)
  • 配置文件语法错误提示到行
  • ...

 

三、

roadhog 从命令的维度看,包含 dev (mock),build,test。

变化是:

  • 原 server 命令改名为 dev
  • dev 和 build 主要是配置上的变化
  • mock 保持不变
  • test 从 mocha 改为 jest,同时支持 UI 组件的测试

 

四、

配置项即功能,这里有比较大的变化。

不再支持的有:

  • multipage (use commons instead)
  • autoprefixer (use browserslist instead)
  • dllPlugin (学习成本太高,并且有其他方法可以提升 dev server 启动速度)
  • svgSpriteLoaderDirs
  • library
  • libraryTarget
  • cssModulesExclude

功能变化的有:

  • publicPath(dev 模式下的也可覆写了)
  • sass(值从 Boolean 改成 Object,内容为传给 node-sass 的配置项)

新的有:

  • babel
  • copy
  • commons
  • browserslist
  • alias (总算加上了,因为我发现 babel 插件代替不了 alias,因为 babel 不处理 node_modules)
  • devtool (可给生产环境开启 sourcemap 了)
  • extraResolveModules
  • manifest(生成 manifest.json)

 

五、

此外,还有一些变化:

  • 不再支持通过 src/index.ejs 的方式生成 Html

 

六、

最后,我们对照 ant-design-pro 的升级 PR 来看下如何升级到 raodhog@2 。

对于大部分的应用来说,只要:

  • 升级 roadhog 版本为 2.x
  • 更名 .roadhogrc 为 .webpackrc
  • 更名 roadhog server 为 roadhog dev

然后,由于内置了 babel@7

  • 如果你有用 babel-polyfill,更换为 @babel/polyfill
  • 如果有 babel-runtime 依赖和配置了 babel-plugin-transform-runtime 插件,需删除,因为已内置处理,否则可能会报 this.setDynamic is not a function 的错误
  • 由于 babel@7 无法使用 babel-plugin-add-module-exports,所有的 require('./file') 需改为 require('./file').default
  • 如果有用 babel-plugin-dva-hmr,请升级到 0.4.x,

再然后,如果有 src/index.ejs,需要额外配置 html 属性,

"html": { "template": "./src/index.ejs" }

最后,有些配置不再支持,需考虑替代方案或使用 webpack.config.js 进行配置。

 

(完)

@afc163
Copy link

afc163 commented Jan 2, 2018

之前有用 src/index.ejs,需移到 public/index.html,并手动引入 CSS 和 JS

如果开启了 hash,这个 html 岂不是要手动维护。

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 2, 2018

这个 public/index.html dev 用(dev 时不会开 hash 的),build 后通常不会用到静态的 html。之前的 html 生成没处理好,很多反馈,就先下掉了,后面后需求再加上。

@afc163
Copy link

afc163 commented Jan 2, 2018

这样会让人误解的,public/index.html 引用的 index.js 在 dist 里根本不存在。

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 2, 2018

那开个 html 的配置项,然后透传给 html-webpack-plugin 吧。

e.g.

"html": {
  "template": "./src/index.ejs"
}

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 2, 2018

umijs/umi#39

@elivoa
Copy link

elivoa commented Jan 3, 2018

“出错点击后跳转到 IDE” 这个 IDE如何设置?

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 3, 2018

“出错点击后跳转到 IDE” 这个 IDE如何设置?

不用设置,自动判断的。

@elivoa
Copy link

elivoa commented Jan 3, 2018

不用设置,自动判断的。

我点击后打开的是sublime text,但是我想让他用web storm打开。是和操作系统的设置有关?比如js文件的默认打开程序?但是我.js默认使用VSCode打开的。

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 3, 2018

你把 sublime 关了,然后开着 webstorm,看下是用什么打开的。

@elivoa
Copy link

elivoa commented Jan 3, 2018

明白了,要设置REACT_EDITOR环境变量

Could not open search.js in the editor.
The editor process exited with an error: spawn code ENOENT.

To set up the editor integration, add something like REACT_EDITOR=atom to the .env.local file in your project folder and restart the development server. Learn more: https://goo.gl/MMTaZt

@henry-fun
Copy link

由于 babel@7 无法使用 babel-plugin-add-module-exports,所有的 require('./file') 需改为 require('./file').default
感觉这个对原项目的改动会很多,暂时还是用1.3.0版本,等以后解决这个问题了再考虑升版本。

@elivoa
Copy link

elivoa commented Jan 3, 2018

dllPlugin (学习成本太高,并且有其他方法可以提升 dev server 启动速度)

有什么办法可以提高dev server速度的么?现在没了dll plugin,启动速度和重新编译速度都很慢。我的项目要半分钟了。原来使用dllplugin的时候是重编速度6-7秒。

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 4, 2018

有什么办法可以提高dev server速度的么?现在没了dll plugin,启动速度和重新编译速度都很慢。我的项目要半分钟了。原来使用dllplugin的时候是重编速度6-7秒。

按需编译,umi 里有实现。

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 4, 2018

感觉这个对原项目的改动会很多,暂时还是用1.3.0版本,等以后解决这个问题了再考虑升版本。

require 用的地方应该不多吧,大部分应该是 import 才对。而且 require().default 是标准的写法,应该不会再回去了,拥抱变化吧,🤗。

@HeskeyBaozi
Copy link

HeskeyBaozi commented Jan 4, 2018

👍 2.0 普天同庆

  • commons 这个选项给了代码块分割手动干预的更多定制
  • definealias 简直是目录结构强迫症的福音
    除了create-react-app外,用来快速迭代,接近0配置的不二选择。

@iceberg211
Copy link

不支持装饰器的写法了吗?

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 4, 2018

不支持装饰器的写法了吗?

要额外引 babel 插件的,1.0 的时候也是这样。

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 4, 2018

commons 可以提高开发时修改之后重新编译的时间么?

不能。

@iceberg211
Copy link

要额外引 babel 插件的,1.0 的时候也是这样。
是babel-plugin-transform-decorators-legacy吧,我在.webpackrc这样配的

  "extraBabelPlugins": [
        "transform-decorators-legacy",
        "lodash", ["import", { "libraryName": "antd-mobile", "style": true }]
    ],

其他的步骤也照做了

@bibiehy
Copy link

bibiehy commented Jan 5, 2018

dva new 创建项目后,手动升级[email protected],严格按照上面修改,但编译失败了。

image
.webpackrc 文件配置:
image

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 5, 2018

@bibiehy 如果有 babel-runtime 依赖和配置了 babel-plugin-transform-runtime 插件,需删除,因为已内置处理,否则可能会报 this.setDynamic is not a function 的错误

@bibiehy
Copy link

bibiehy commented Jan 5, 2018

@sorrycc 谢谢,2个都删除掉,修改"extraBabelPlugins": ["@babel/transform-runtime"]就好了。

还一个问题,js 校验工具ESLint 的配置在哪里修改或者禁用。

@+babel/transform-runtime不见了。|- _ -

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 5, 2018

可以 DISABLE_ESLINT roadhog dev 禁用,修改的话本地配 .eslintrc 就好了。

@Tsury
Copy link

Tsury commented Jan 6, 2018

No more --watch for roadhog build? I need that...
I'm building for a non-browser app and I need to build on every change. The watch option made it very fast. Now if I set a watch task, it rebuilds everything on every change, which is too slow for me.

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 7, 2018

@Tsury thanks for feedback, it's supported in [email protected] .

@Tsury
Copy link

Tsury commented Jan 8, 2018

@sorrycc thanks! Another request - is it possible to build in debug? I'm trying to build, but it only builds for production.

I see this line in af-webpack build.js line 32:
process.env.NODE_ENV = 'production'; // These sizes are pretty large. We'll warn for bundles exceeding them.

Then in af-webpack's getConfig.js file line 66:
const isDev = process.env.NODE_ENV === 'development';

And then everything in the config is optimized for release (uglified, etc...) because isDev === false.

Is there a way to build debug output files (no dev server)? No compression, no optimizations.

@sorrycc
Copy link
Owner Author

sorrycc commented Jan 8, 2018

@Tsury No compression is supported by NO_COMPRESS=1 roadhog build.

@Tsury
Copy link

Tsury commented Jan 8, 2018

It's still not enough.
There are properties in the config that are determined solely by 'isDev', which is false. When I set NO_COMPRESS=1, my binary get's just a little bigger (~1.7mb), but it still very optimized. When I force isDev === true, the binary get's huge (5.5mb), and there's no compression.

Also, another issue - because of:
process.env.NODE_ENV = 'production'; // These sizes are pretty large. We'll warn for bundles exceeding them.

This line forces my ENV to be production, and then my 'production' configuration in .webpackrc.js is used, instead of my development configuration.

Something here looks a little awkward to me.

EDIT: I would just like to control the NODE_ENV myself, and build debug, without af-webpack forcing it to be 'production'...
EDIT 2: This forcing of NODE_ENV will also make React run in production mode. To summarize - I can't currently use roadhog build and have NODE_ENV = 'development'
LAST EDIT: I believe all of this is caused because now roadhog build --debug was removed.

@feitingjun
Copy link

不知道为什么,打包后的在ie里面显示不出来,包括ie11,空白的什么都没有,

tim
就报了个这个错,这是什么情况呢?

@feitingjun
Copy link

部署到服务器和一样的问题

@sorrycc
Copy link
Owner Author

sorrycc commented May 14, 2018

@feitingjun 出错信息见 https://reactjs.org/docs/error-decoder.html?invariant=143 ,不压缩跑跑看。

@feitingjun
Copy link

在开发环境跑是没有问题的,也没有警告,不用dva直接在入口index.js文件中写ReactDom都出不来,而且在其他浏览器也是没有问题的

@xiangweijun
Copy link

xiangweijun commented May 21, 2018

roadhog升级至2.4.1后,配置文件webpackrc里,env下的production配置为什么不起作用了吗,
env: {
production: {
outputPath: path.join(__dirname, dist/1.0.0),
}
}
为什么正式环境打包后,输出目录是dist,而不是dist/1.0.0 @sorrycc

@xieyunjie
Copy link

你升级到最新版本就可以了。

@dsmelon
Copy link

dsmelon commented Jun 4, 2018

那个commons的配置可不可以写的更清楚一些呢,比如我想把moment库提出来

@99874405
Copy link

99874405 commented Jun 8, 2018

roadhog 建构完的包很大,怎样使用tree-shaking

@big-tutu
Copy link

"dllPlugin (学习成本太高,并且有其他方法可以提升 dev server 启动速度)", 这里的其他方法,还请给一些指导,谢谢!

@SherryJiang1130
Copy link

不能打包多页面入口文件吗?这样配置build之后只有一个html文件,下面html,也不能配置两个.ejs文件,需要打包出两个html文件,怎么搞?
image

@capaten
Copy link

capaten commented Aug 10, 2018

@sorrycc 您好,我刚接触这个,不是太懂,想请教您一下:我想使用antd-theme-webpack-plugin插件来修改主题,但不知道该怎么配置,文档里给的使用方法如下:
`const themePlugin = new AntDesignThemePlugin(options);
// in config object
plugins: [
themePlugin
]

Add this plugin in `plugins` array.`
但我的.webpackrc.js配置文件里没有plugins,内容如下:
`export default {
  entry: 'src/index.js',
  extraBabelPlugins: [
    "@babel/plugin-transform-runtime",
    ['import', { libraryName: 'antd', libraryDirectory: 'es', style: true }],
  ],
  env: {
    development: {
      extraBabelPlugins: ['dva-hmr'],
    }
  },

  externals: {
    '@antv/data-set': 'DataSet',
    rollbar: 'rollbar',
  },
  alias: {
    components: path.resolve(__dirname, 'src/components/'),
  },
  ignoreMomentLocale: true,
  theme: './src/theme.js',
  html: {
    template: './src/index.ejs',
  },
  lessLoaderOptions: {
    javascriptEnabled: true,
  },
  disableDynamicImport: true,
  disableCSSModules:false,
  publicPath: '/',
  hash: true,
};`
请问themePlugin该放在哪个里面?

@micheal-zhibin
Copy link

micheal-zhibin commented Aug 23, 2018

在index.js中引用了sass文件
import './css/index.scss';
用的html模板index.ejs
编译之后看到html多出一个style标签,里面存的都是我的sass文件内容而没有编译过
我的配置如下:
{
"entry": "./src/index.js",
"define": {
"process.env.NODE_ENV": "development"
},
"html": {
"template": "./src/index.template.ejs"
},
"disableCSSModules": true
}
我配置中加上sass也没用,配置如下:
{
"entry": "./src/index.js",
"define": {
"process.env.NODE_ENV": "development"
},
"html": {
"template": "./src/index.template.ejs"
},
"disableCSSModules": true,
"sass": {
"file": "./src/css/index.scss"
}
}

@xu-Aurora
Copy link

如果你有用 babel-polyfill,更换为 @babel/polyfill如果有 babel-runtime 依赖和配置了 babel-plugin-transform-runtime 插件,需删除,因为已内置处理,否则可能会报 this.setDynamic is not a function 的错误由于 babel@7 无法使用 babel-plugin-add-module-exports,所有的 require('./file') 需改为 require('./file').default如果有用 babel-plugin-dva-hmr,请升级到 0.4.x,发自我的华为手机-------- 原始邮件 --------主题:Re: [sorrycc/blog] roadhog 2.0 发布 (#55)发件人:Pecking 收件人:sorrycc/blog 抄送:zhouyi [email protected],Mention 这样手动删除了babel-plugin-transform-runtime,他会报错不存在这个依赖;怎么解决了? —You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread. {"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/sorrycc/blog","title":"sorrycc/blog","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"[https://github.com/sorrycc/blog"}},"updates":{"snippets":{"icon":"PERSON","message":"@Pecking in #55: jzo4 v xk 5 mo0j k67\r\nfqr%5Cr%5Cn!%5Bfqr) 6d91dskz_cvwt w d]([https://user-images.githubusercontent.com/27414779/39610962-7511ab90-4f86-11e8-8e44-db64e50cd849.png)\r\n这样手动删除了babel-plugin-transform-runtime,他会报错不存在这个依赖;怎么解决了?"}],"action":{"name":"View](https://user-images.githubusercontent.com/27414779/39610962-7511ab90-4f86-11e8-8e44-db64e50cd849.png)%5Cr%5Cn%E8%BF%99%E6%A0%B7%E6%89%8B%E5%8A%A8%E5%88%A0%E9%99%A4%E4%BA%86babel-plugin-transform-runtime,%E4%BB%96%E4%BC%9A%E6%8A%A5%E9%94%99%E4%B8%8D%E5%AD%98%E5%9C%A8%E8%BF%99%E4%B8%AA%E4%BE%9D%E8%B5%96;%E6%80%8E%E4%B9%88%E8%A7%A3%E5%86%B3%E4%BA%86%EF%BC%9F%22%7D%5D,%22action%22:%7B%22name%22:%22View) Issue","url":"#55 (comment)"}}}

按你说的做了还是报了this.setDynamic is not a function的错,根本就没有解决

@pengshaosu
Copy link

roadhog 建构完的包很大,怎样使用tree-shaking

roadhog2.0这个版本支持webpack-deep-scope-plugin?

@gggdandan
Copy link

"build": "cross-env NODE_ENV=prd roadhog build"这个命令在文件中判断环境 process.env.NODE_ENV的值怎么不是prd

@gggdandan
Copy link

“build”:“cross-env NODE_ENV = prd roadhog build”这个命令在文件中判断环境process.env.NODE_ENV的值怎么不是prd

sorrycc/roadhog#271

@zhangpanfei
Copy link

define 用不了

@jeekliang
Copy link

DISABLE_ESLINT

可以 DISABLE_ESLINT roadhog dev 禁用,修改的话本地配 .eslintrc 就好了。

how to disable eslint in [email protected] ?

@tang-yue
Copy link

要额外引 babel 插件的,1.0 的时候也是这样。
是babel-plugin-transform-decorators-legacy吧,我在.webpackrc这样配的

  "extraBabelPlugins": [
        "transform-decorators-legacy",
        "lodash", ["import", { "libraryName": "antd-mobile", "style": true }]
    ],

其他的步骤也照做了

请问你这样配了transform-decorators-legacy 生效了吗,为什么我的不生效呢。

@tang-yue
Copy link

export default {
entry: "src/index.js",
extraBabelPlugins: [
"transform-decorators-legacy",
["import", { libraryName: "antd", libraryDirectory: "es", style: true }],
]
}

这样配transform-decorators-legacy 不生效。

@Njuhobby
Copy link

Njuhobby commented May 5, 2019

要额外引 babel 插件的,1.0 的时候也是这样。
是babel-plugin-transform-decorators-legacy吧,我在.webpackrc这样配的

"extraBabelPlugins": [
"transform-decorators-legacy",
"lodash", ["import", { "libraryName": "antd-mobile", "style": true }]
],

其他的步骤也照做了

不知道你后来配置好了吗?我在我的webpackrc file中试了如下的配置:

extraBabelPlugins: [ ["@babel/plugin-proposal-decorators", { "legacy": true }], ["@babel/plugin-proposal-class-properties", { "loose": true }], ['import', { libraryName: 'antd', libraryDirectory: 'es', style: true }] ],

但还是报错说 “Error: Cannot use the decorators and decorators-legacy plugin together”

@xueyanhui123456
Copy link

为什么配置了代理但是不管用

@xueyanhui123456
Copy link

为什么设置代理不管用

@Shen-Chris
Copy link

添加某些第三方依赖(eg griffith,yarn add griffith),其有es6语法,利用 extraBabelIncludes: [path.resolve(__dirname, './node_modules/griffith')] 或alias 都无法 转换es5 语法,使得build最后压缩报错:Failed to minify the bundle. Error: index.js from UglifyJs
Unexpected token: punc (() / ?;怎么破?

@Shen-Chris
Copy link

好吧extraBabelIncludes不用path.resolve,extraBabelIncludes:['./node_modules/griffith','...']

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests