Lint

.eslintrc.js

@@ -0,0 +1,22 @@
+module.exports = {
+  root: true,
+  parser: 'babel-eslint',
+  parserOptions: {
+    sourceType: 'module'
+  },
+  // https://github.com/feross/standard/blob/master/RULES.md#javascript-standard-style
+  extends: 'standard',
+  // required to lint *.vue files
+  plugins: [
+    'html'
+  ],
+  // add your custom rules here
+  'rules': {
+    // allow paren-less arrow functions
+    'arrow-parens': 0,
+    // allow async-await
+    'generator-star-spacing': 0,
+    // allow debugger during development
+    'no-debugger': process.env.NODE_ENV === 'production' ? 2 : 0
+  }
+}

.travis.yml

@@ -4,6 +4,7 @@ node_js:
 branches:
   only: master
 script:
+  - npm run test
   - npm run build:docs
 after_script:
   - cp ./docs/favicon.png ./dist/

build/webpack.docs.config.js

@@ -1,5 +1,4 @@
 const path = require('path')
-const webpack = require('webpack')
 const HtmlWebpackPlugin = require('html-webpack-plugin')
 
 module.exports = {
@@ -12,12 +11,12 @@ module.exports = {
     rules: [
       {
         test: /\.vue$/,
-        use: ['vue-loader'],
+        use: ['vue-loader', 'eslint-loader'],
         exclude: /_cache/
       },
       {
         test: /\.js$/,
-        use: ['babel-loader'],
+        use: ['babel-loader', 'eslint-loader'],
         exclude: [/node_modules/, /md/]
       },
       {
@@ -45,13 +44,13 @@ module.exports = {
   },
   resolve: {
     alias: {
-      '@': __dirname + '/../src',
-      docs: __dirname + '/../docs/src',
+      '@': path.resolve(__dirname, '../src'),
+      docs: path.resolve(__dirname, '../docs/src')
     }
   },
   plugins: [
     new HtmlWebpackPlugin({
-      template: './docs/template/index.html'
+      template: path.resolve(__dirname, '../docs/template/index.html')
     })
   ]
 }

build/webpack.prod.config.js

@@ -1,5 +1,4 @@
 const path = require('path')
-const webpack = require('webpack')
 
 module.exports = {
   entry: './src/index.js',
@@ -24,7 +23,7 @@ module.exports = {
   },
   resolve: {
     alias: {
-      '@': __dirname + '/../src'
+      '@': path.resolve(__dirname, '../src')
     }
-  },
+  }
 }

docs/components/App.vue

@@ -99,4 +99,4 @@ h2 {
   color: #3f51b5;
   border-right: 2px solid #3f51b5;
 }
-</style>
+</style>

docs/components/CateView.vue

@@ -1,3 +1,3 @@
 <template lang="pug">
 router-view
-</template>
+</template>

docs/components/DocPreview.vue

@@ -31,4 +31,4 @@
 .mdl-textfield {
   width: 100%;
 }
-</style>
+</style>

docs/components/Drawer.vue

@@ -1,4 +1,4 @@
 <template lang="pug">
 nav.mdl-navigation
   slot
-</template>
+</template>

docs/components/RootFrame.vue

@@ -43,7 +43,7 @@ export default {
   },
   methods: {
     close () {
-      document.querySelector('.mdl-layout__obfuscator').classList.remove('is-visible');
+      document.querySelector('.mdl-layout__obfuscator').classList.remove('is-visible')
       this.$refs.drawer.classList.remove('is-visible')
     }
   },
@@ -55,4 +55,4 @@ export default {
     })
   }
 }
-</script>
+</script>

docs/main.js

@@ -14,25 +14,25 @@ import 'material-design-lite/material.min.js'
 import 'material-design-icons/iconfont/material-icons.css'
 import './fonts/iconfont.css'
 
-Vue.use(VueRouter);
+Vue.use(VueRouter)
 Vue.use(BaiduMap, {
   ak: 'oW2UEhdth2tRbEE4FUpF9E5YVDCIPYih'
 })
 
 Vue.component('doc-preview', DocPreview)
 
 const router = new VueRouter({
-    routes,
-    linkActiveClass: 'is-active',
+  routes,
+  linkActiveClass: 'is-active'
 })
 
 router.afterEach(route => {
   let blocks = document.querySelectorAll('pre code')
   Array.prototype.forEach.call(blocks, Hljs.highlightBlock)
 })
 
-new Vue({
+export default new Vue({
   el: '#app',
   router,
   render: h => h(App)
-})
+})

docs/routes.js

@@ -38,7 +38,7 @@ export default [
   {
     // 首页
     path: '/',
-    name:  'VUE BAIDU MAP',
+    name: 'VUE BAIDU MAP',
     component: DocIndex,
     meta: 'hidden'
   },
@@ -79,7 +79,7 @@ export default [
         path: 'bm-view',
         component: DocBmView,
         name: '地图视图'
-      },
+      }
     ]
   },
   {
@@ -214,7 +214,7 @@ export default [
         path: 'item',
         name: '菜单项',
         component: DocContextMenuItem
-      },
+      }
     ]
   }
-]
+]

package.json

@@ -10,7 +10,8 @@
     "build:docs": "webpack -p --config ./build/webpack.docs.config.js",
     "build": "webpack -p --config ./build/webpack.prod.config.js",
     "dev": "webpack-dev-server --content-base docs --config ./build/webpack.docs.config.js --hot --inline",
-    "deploy": "npm run build && npm run build:docs"
+    "deploy": "npm run build && npm run build:docs",
+    "test": "eslint --ext .js,.vue src docs build"
   },
   "repository": {
     "type": "git",
@@ -32,12 +33,21 @@
   "homepage": "https://github.com/Dafrok/vue-baidu-map#readme",
   "devDependencies": {
     "babel-core": "^6.21.0",
+    "babel-eslint": "^7.2.3",
     "babel-loader": "^6.2.10",
     "babel-plugin-add-module-exports": "^0.2.1",
     "babel-plugin-transform-es2015-modules-umd": "^6.24.0",
     "babel-preset-es2015": "^6.18.0",
     "bulma": "^0.3.0",
     "css-loader": "^0.26.1",
+    "eslint": "^3.19.0",
+    "eslint-config-standard": "^10.2.1",
+    "eslint-loader": "^1.7.1",
+    "eslint-plugin-html": "^2.0.1",
+    "eslint-plugin-import": "^2.2.0",
+    "eslint-plugin-node": "^4.2.2",
+    "eslint-plugin-promise": "^3.5.0",
+    "eslint-plugin-standard": "^3.0.1",
     "file-loader": "^0.10.0",
     "font-awesome": "^4.7.0",
     "github-markdown-css": "^2.4.1",

src/base/bindEvent.js

@@ -6,4 +6,3 @@ export default function (instance) {
     })
   })
 }
-

src/base/events.js

@@ -108,4 +108,4 @@ export default {
     'click',
     'dblclick'
   ]
-}
+}

src/base/factory.js

@@ -1,24 +1,24 @@
-export function createPoint(BMap, options = {}) {
+export function createPoint (BMap, options = {}) {
   const {lng, lat} = options
   return new BMap.Point(parseFloat(lng), parseFloat(lat))
 }
 
-export function createPixel(BMap, options = {}) {
+export function createPixel (BMap, options = {}) {
   const {x, y} = options
   return new BMap.Pixel(parseFloat(x), parseFloat(y))
 }
 
-export function createBounds(BMap, options = {}) {
+export function createBounds (BMap, options = {}) {
   const {sw, ne} = options
   return new BMap.Bounds(createPoint(BMap, sw), createPoint(BMap, ne))
 }
 
-export function createSize(BMap, options = {}) {
+export function createSize (BMap, options = {}) {
   const {width, height} = options
   return new BMap.Size(parseFloat(width), parseFloat(height))
 }
 
-export function createIcon(BMap, options = {}) {
+export function createIcon (BMap, options = {}) {
   const {url, size, opts = {}} = options
   return new BMap.Icon(url, createSize(BMap, size), {
     anchor: opts.anchor && createSize(BMap, opts.anchor),
@@ -35,4 +35,4 @@ export function createLabel (BMap, options = {}) {
     position: opts.position && createPoint(BMap, opts.position),
     enableMassClear: opts.enableMassClear
   })
-}
+}

src/base/mixins/common.js

@@ -35,13 +35,13 @@ class Mixin {
         this.$nextTick(() => {
           const {map, originInstance} = this
           try {
-            map[types[prop.type].unload](this.originInstance)
+            map[types[prop.type].unload](originInstance)
           } catch (e) {}
         })
       }
     }
     this.mounted = function () {
-      const {BMap, map} = this.$parent
+      const {map} = this.$parent
       const {ready} = this
       map ? ready() : this.$parent.$on('ready', ready)
     }
@@ -51,4 +51,4 @@ class Mixin {
   }
 }
 
-export default type => new Mixin({type})
+export default type => new Mixin({type})

src/context-menu/Item.vue

@@ -8,7 +8,7 @@ export default {
   props: {
     callback: {
       type: Function,
-      default: new Function
+      default: function () {}
     },
     text: {
       type: String
@@ -58,4 +58,4 @@ export default {
     this.reload()
   }
 }
-</script>
+</script>

src/context-menu/Menu.vue

@@ -33,7 +33,7 @@ export default {
             target: parent
           })
         }, {
-          width: this.width,
+          width,
           id: item.id,
           iconUrl: item.iconUrl
         })
@@ -44,4 +44,4 @@ export default {
     }
   }
 }
-</script>
+</script>

src/controls/CityList.vue

@@ -3,13 +3,11 @@ import commonMixin from '@/base/mixins/common.js'
 
 export default {
   name: 'bm-city-list',
-  render () {
-    return
-  },
+  render () {},
   mixins: [commonMixin('control')],
   props: {
     anchor: {
-      type: String,
+      type: String
     },
     offset: {
       type: Object
@@ -40,4 +38,4 @@ export default {
     }
   }
 }
-</script>
+</script>

src/controls/Copyright.vue

@@ -3,9 +3,7 @@ import commonMixin from '@/base/mixins/common.js'
 
 export default {
   name: 'bm-copyright',
-  render () {
-    return
-  },
+  render () {},
   mixins: [commonMixin('control')],
   props: ['anchor', 'offset', 'copyright'],
   watch: {
@@ -34,7 +32,7 @@ export default {
       const {removeCopyright, getCopyrightCollection} = this.originInstance
       const copyrightList = getCopyrightCollection()
       copyrightList && copyrightList.forEach(item => {
-        this.originInstance.removeCopyright(item.id)
+        removeCopyright(item.id)
       })
       this.copyright && this.copyright.forEach(item => {
         const bounds = item.bounds
@@ -50,4 +48,4 @@ export default {
     }
   }
 }
-</script>
+</script>

src/controls/Geolocation.vue

@@ -4,13 +4,11 @@ import bindEvents from '@/base/bindEvent.js'
 
 export default {
   name: 'bm-geolocation',
-  render () {
-    return
-  },
+  render () {},
   mixins: [commonMixin('control')],
   props: {
     anchor: {
-      type: String,
+      type: String
     },
     offset: {
       type: Object
@@ -56,4 +54,4 @@ export default {
     }
   }
 }
-</script>
+</script>