Back

eslint 和 jslint: 检查的太严格了,没太大意义.

发布时间: 2016-07-29 10:37:00

先说结语: 决定代码质量的,是人.而不是工具.  良好的命名,清晰的思路, 那就是好代码.  混乱的思路, 代码再好看,命名搞错,也是烂代码. 

而且不同的代码风格, 都有自己存在的合理性. 没有必要为了一个代码风格, 花大力气去学习他们的配置文件.

最近,项目中在使用 eslint, 搞得我们欲仙欲死. 

多个空格, 报错.  少个空格,报错. 定义个方法没用,报错. 各种奇怪的位置,要有空格. 没有,就报错. 

Screenshot From 2016 07 29 18 41 55

Screenshot From 2016 07 29 18 40 39

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",

Back