Add eslint

Author: pspgbhu

.babelrc

@@ -1,10 +1,5 @@
 {
   "presets": ["es2015", "stage-2"],
   "plugins": ["transform-runtime"],
-  "comments": false,
-  "env": {
-    "test": {
-      "plugins": [ "istanbul" ]
-    }
-  }
+  "comments": false
 }

.eslintrc.js

@@ -0,0 +1,46 @@
+module.exports = {
+  env: {
+    "browser": true,
+  },
+  root: true,
+  parser: 'babel-eslint',
+  parserOptions: {
+    sourceType: 'module'
+  },
+  extends: 'airbnb-base',
+  // required to lint *.vue files
+  plugins: [
+    'html'
+  ],
+  // check if imports actually resolve
+  'settings': {
+    'import/resolver': {
+      'webpack': {
+        'config': 'build/webpack.base.conf.js'
+      }
+    }
+  },
+  // add your custom rules here
+  'rules': {
+    // don't require .vue extension when importing
+    'import/extensions': ['error', 'always', {
+      'js': 'never',
+      'vue': 'never'
+    }],
+    'import/no-dynamic-require': 0,
+    // allow debugger during development
+    'no-debugger': process.env.NODE_ENV === 'production' ? 2 : 0,
+    "no-use-before-define": ["error", { "functions": false,}],
+    "no-unused-vars": 0,
+
+    "no-console": 0,
+    "no-underscore-dangle": 0,
+    "arrow-parens": 0,
+    "no-param-reassign": 0,
+    "no-unused-expressions": 0,
+    "padded-blocks": 0,
+    "prefer-const": 1,
+    "linebreak-style":0,
+    "max-len":0
+  }
+}

build/build.js

@@ -10,13 +10,6 @@ var webpack = require('webpack')
 
 var webpackConfig = require('./webpack.prod.conf');
 
-if (env.NODE_ENV_EXAMPLE === 'y') {
-  webpackConfig.entry = {
-    main: './example/main.js'
-  };
-  env.NODE_ENV_EXAMPLE === 'n';
-}
-
 console.log(
   '  Tip:\n' +
   '  Built files are meant to be served over an HTTP server.\n' +

build/webpack.base.conf.js

@@ -12,6 +12,9 @@ var cssSourceMapProd = (env === 'production' && config.build.productionSourceMap
 var useCssSourceMap = cssSourceMapDev || cssSourceMapProd
 
 module.exports = {
+  entry: {
+    main: './example/main.js'
+  },
   output: {
     path: config.build.assetsRoot,
     publicPath: process.env.NODE_ENV === 'production' ? config.build.assetsPublicPath : config.dev.assetsPublicPath,
@@ -32,6 +35,26 @@ module.exports = {
     fallback: [path.join(__dirname, '../node_modules')]
   },
   module: {
+    preLoaders: [
+      {
+        test: /\.vue$/,
+        loader: 'eslint',
+        include: [
+          path.join(projectRoot, 'src'),
+          path.join(projectRoot, 'example')
+        ],
+        exclude: /node_modules/
+      },
+      {
+        test: /\.js$/,
+        loader: 'eslint',
+        include: [
+          path.join(projectRoot, 'src'),
+          path.join(projectRoot, 'example')
+        ],
+        exclude: /node_modules/
+      }
+    ],
     loaders: [
       {
         test: /\.vue$/,
@@ -68,6 +91,9 @@ module.exports = {
       }
     ]
   },
+  eslint: {
+    formatter: require('eslint-friendly-formatter')
+  },
   vue: {
     loaders: utils.cssLoaders({ sourceMap: useCssSourceMap }),
     postcss: [

build/webpack.dev.conf.js

@@ -7,14 +7,11 @@ var HtmlWebpackPlugin = require('html-webpack-plugin')
 var FriendlyErrors = require('friendly-errors-webpack-plugin')
 
 // add hot-reload related code to entry chunks
-// Object.keys(baseWebpackConfig.entry).forEach(function (name) {
-//   baseWebpackConfig.entry[name] = ['./build/dev-client'].concat(baseWebpackConfig.entry[name])
-// })
+Object.keys(baseWebpackConfig.entry).forEach(function (name) {
+  baseWebpackConfig.entry[name] = ['./build/dev-client'].concat(baseWebpackConfig.entry[name])
+})
 
 module.exports = merge(baseWebpackConfig, {
-  entry: {
-    main: './example/main.js'
-  },
   module: {
     loaders: utils.styleLoaders({ sourceMap: config.dev.cssSourceMap })
   },

build/webpack.prod.conf.js

@@ -9,9 +9,6 @@ var HtmlWebpackPlugin = require('html-webpack-plugin')
 var env = config.build.env
 
 var webpackConfig = merge(baseWebpackConfig, {
-  entry: {
-    'swipe': './src/main.js',
-  },
   module: {
     loaders: utils.styleLoaders({ sourceMap: config.build.productionSourceMap, extract: true })
   },
@@ -102,4 +99,8 @@ if (config.build.productionGzip) {
   )
 }
 
+delete webpackConfig.entry;
+webpackConfig.entry = {
+  'swipe': './src/main.js',
+}
 module.exports = webpackConfig

dist/swipe.js

@@ -19,8 +19,8 @@ export default {
   mounted() {
     document.getElementById('button').onclick = () => {
       this.index = this.index === 1 ? 0 : this.index + 1;
-    }
-  }
+    };
+  },
 };
 </script>
 

example/App.vue

@@ -1,16 +1,31 @@
 import Vue from 'vue';
+import { Swipe, SwipeItem } from '../src/main';
+// import { Swipe, SwipeItem } from 'c-swipe';
 import App from './App';
-import dist from '../dist/swipe.js'
 
-import 'c-swipe/dist/swipe.css';
-import { Swipe, SwipeItem } from 'c-swipe';
-// import { Swipe, SwipeItem } from '../src/main.js';
+// require('c-swipe/dist/swipe.css');
+
+console.log(process.env);
+
+/* eslint-disable global-require */
+// if (process.env === 'test') {
+//   require('c-swipe/dist/swipe.css');
+// }
+
+/* eslint-disable global-require */
+// const Swipe = process.env === 'test'
+//   ? require('../dist/swipe').Swipe
+//   : require('../src/main').Swipe;
+//
+// const SwipeItem = process.env === 'test'
+//   ? require('../dist/swipe').SwipeItem
+//   : require('../src/main').SwipeItem;
 
 Vue.component('swipe', Swipe);
 Vue.component('swipe-item', SwipeItem);
 
 const vm = new Vue({
   el: '#app',
   template: '<App/>',
-  components: { App }
+  components: { App },
 });