eslint 和 jslint: 检查的太严格了,没太大意义.
访问量: 5855
先说结语: 决定代码质量的,是人.而不是工具. 良好的命名,清晰的思路, 那就是好代码. 混乱的思路, 代码再好看,命名搞错,也是烂代码.
而且不同的代码风格, 都有自己存在的合理性. 没有必要为了一个代码风格, 花大力气去学习他们的配置文件.
最近,项目中在使用 eslint, 搞得我们欲仙欲死.
多个空格, 报错. 少个空格,报错. 定义个方法没用,报错. 各种奇怪的位置,要有空格. 没有,就报错.
checkstyle出现那年, 还没jslint呢.
我作为checkstyle国内的第一个译者, 到现在也没发现checkstyle 多大用.
果断取消掉.
取消的步骤:
1. 去掉 build/webpack.base.conf.js 中的所有 eslint 代码:
@@ -25,20 +25,6 @@ module.exports = { fallback: [path.join(__dirname, '../node_modules')] }, module: { - preLoaders: [ - { - test: /\.vue$/, - loader: 'eslint', - include: projectRoot, - exclude: /node_modules/ - }, - { - test: /\.js$/, - loader: 'eslint', - include: projectRoot, - exclude: /node_modules/ - } - ], loaders: [ { test: /\.vue$/, @@ -74,8 +60,5 @@ module.exports = { }, vue: { loaders: cssLoaders() - }, - eslint: { - formatter: require('eslint-friendly-formatter') } }
2. 去掉 package.js 中所有的 eslint:
- "eslint": "2.11.0", - "eslint-config-standard": "5.3.1", - "eslint-friendly-formatter": "1.2.2", - "eslint-loader": "1.3.0", - "eslint-plugin-html": "1.5.1", - "eslint-plugin-promise": "^1.0.8", - "eslint-plugin-standard": "^1.3.2",