Add eslint

.eslintrc.js

@@ -0,0 +1,205 @@
+module.exports = {
+    "env": {
+        "browser": true,
+        "commonjs": true
+    },
+    "extends": "eslint:recommended",
+    "rules": {
+        "accessor-pairs": 2,
+        "array-bracket-spacing": [
+            2,
+            "never"
+        ],
+        "array-callback-return": 2,
+        "arrow-body-style": 2,
+        "arrow-parens": 2,
+        "arrow-spacing": 2,
+        "block-scoped-var": 2,
+        "block-spacing": 2,
+        "brace-style": 2,
+        "callback-return": 2,
+        "camelcase": 2,
+        "comma-spacing": [
+            2,
+            {
+                "after": true,
+                "before": false
+            }
+        ],
+        "comma-style": [
+            2,
+            "last"
+        ],
+        "complexity": 2,
+        "computed-property-spacing": 2,
+        "consistent-return": 2,
+        "consistent-this": 2,
+        "curly": 0,
+        "default-case": 2,
+        "dot-location": 2,
+        "dot-notation": 2,
+        "eol-last": 2,
+        "eqeqeq": 2,
+        "func-names": 0,
+        "func-style": 2,
+        "generator-star-spacing": 2,
+        "global-require": 2,
+        "guard-for-in": 2,
+        "handle-callback-err": 2,
+        "id-blacklist": 2,
+        "id-length": 0,
+        "id-match": 2,
+        "indent": 2,
+        "init-declarations": 2,
+        "jsx-quotes": 2,
+        "key-spacing": 2,
+        "keyword-spacing": [
+            2,
+            {
+                "after": true,
+                "before": true
+            }
+        ],
+        "linebreak-style": [
+            2,
+            "unix"
+        ],
+        "lines-around-comment": 2,
+        "max-depth": 2,
+        "max-len": 2,
+        "max-nested-callbacks": 2,
+        "max-params": 2,
+        "max-statements": 2,
+        "new-cap": 2,
+        "new-parens": 2,
+        "newline-after-var": [
+            2,
+            "always"
+        ],
+        "newline-per-chained-call": 0,
+        "no-alert": 2,
+        "no-array-constructor": 2,
+        "no-bitwise": 2,
+        "no-caller": 2,
+        "no-catch-shadow": 2,
+        "no-confusing-arrow": 2,
+        "no-continue": 2,
+        "no-div-regex": 2,
+        "no-else-return": 2,
+        "no-empty-function": 2,
+        "no-eq-null": 2,
+        "no-eval": 2,
+        "no-extend-native": 2,
+        "no-extra-bind": 2,
+        "no-extra-label": 2,
+        "no-extra-parens": 2,
+        "no-floating-decimal": 2,
+        "no-implicit-coercion": 2,
+        "no-implicit-globals": 2,
+        "no-implied-eval": 2,
+        "no-inline-comments": 2,
+        "no-invalid-this": 2,
+        "no-iterator": 2,
+        "no-label-var": 2,
+        "no-labels": 2,
+        "no-lone-blocks": 2,
+        "no-lonely-if": 2,
+        "no-loop-func": 2,
+        "no-magic-numbers": 2,
+        "no-mixed-requires": 2,
+        "no-multi-spaces": 2,
+        "no-multi-str": 2,
+        "no-multiple-empty-lines": 2,
+        "no-native-reassign": 2,
+        "no-negated-condition": 2,
+        "no-nested-ternary": 2,
+        "no-new": 2,
+        "no-new-func": 2,
+        "no-new-object": 2,
+        "no-new-require": 2,
+        "no-new-wrappers": 2,
+        "no-octal-escape": 2,
+        "no-param-reassign": 0,
+        "no-path-concat": 2,
+        "no-plusplus": 2,
+        "no-process-env": 2,
+        "no-process-exit": 2,
+        "no-proto": 2,
+        "no-restricted-imports": 2,
+        "no-restricted-modules": 2,
+        "no-restricted-syntax": 2,
+        "no-return-assign": 2,
+        "no-script-url": 2,
+        "no-self-compare": 2,
+        "no-sequences": 2,
+        "no-shadow": 2,
+        "no-shadow-restricted-names": 2,
+        "no-spaced-func": 2,
+        "no-sync": 2,
+        "no-ternary": 2,
+        "no-throw-literal": 2,
+        "no-trailing-spaces": 2,
+        "no-undef-init": 2,
+        "no-undefined": 2,
+        "no-underscore-dangle": 2,
+        "no-unmodified-loop-condition": 2,
+        "no-unneeded-ternary": 2,
+        "no-unused-expressions": 2,
+        "no-use-before-define": 2,
+        "no-useless-call": 2,
+        "no-useless-concat": 2,
+        "no-useless-constructor": 2,
+        "no-var": 0,
+        "no-void": 2,
+        "no-warning-comments": 2,
+        "no-whitespace-before-property": 2,
+        "no-with": 2,
+        "object-curly-spacing": 2,
+        "object-shorthand": 0,
+        "one-var": 0,
+        "one-var-declaration-per-line": 2,
+        "operator-assignment": 2,
+        "operator-linebreak": 2,
+        "padded-blocks": [
+            2,
+            "never"
+        ],
+        "prefer-arrow-callback": 0,
+        "prefer-const": 2,
+        "prefer-reflect": 2,
+        "prefer-rest-params": 2,
+        "prefer-spread": 2,
+        "prefer-template": 2,
+        "quote-props": 0,
+        "quotes": [
+            2,
+            "single"
+        ],
+        "radix": 2,
+        "require-jsdoc": 2,
+        "require-yield": 2,
+        "semi": 2,
+        "semi-spacing": 2,
+        "sort-imports": 2,
+        "sort-vars": 2,
+        "space-before-blocks": 2,
+        "space-before-function-paren": 0,
+        "space-in-parens": [
+            2,
+            "never"
+        ],
+        "space-infix-ops": 2,
+        "space-unary-ops": 2,
+        "spaced-comment": 2,
+        "strict": [
+            2
+        ],
+        "template-curly-spacing": 2,
+        "valid-jsdoc": 2,
+        "vars-on-top": 0,
+        "wrap-iife": 2,
+        "wrap-regex": 2,
+        "yield-star-spacing": 2,
+        "yoda": 2
+    }
+};

index.js

@@ -1,9 +1,10 @@
+/* global L */
+'use strict';
 var gju = require('geojson-utils');
 
 var leafletPip = {
     bassackwards: false,
     pointInLayer: function(p, layer, first) {
-        'use strict';
         if (p instanceof L.LatLng) p = [p.lng, p.lat];
         else if (leafletPip.bassackwards) p = p.concat().reverse();
 

package.json

@@ -4,7 +4,7 @@
   "description": "point in polygon searches for leaflet",
   "main": "index.js",
   "scripts": {
-    "test": "browserify test/*.js | tape-run",
+    "test": "eslint index.js && browserify test/*.js | tape-run",
     "build": "browserify -s leafletPip index.js > leaflet-pip.js && uglifyjs leaflet-pip.js -c -m > leaflet-pip.min.js"
   },
   "keywords": [
@@ -21,11 +21,12 @@
     "url": "git://github.com/mapbox/leaflet-pip.git"
   },
   "devDependencies": {
-    "tape": "~2.13.3",
-    "tape-run": "~0.1.1",
+    "browserify": "~4.1.11",
+    "eslint": "^2.2.0",
     "leaflet": "~0.7.3",
     "phantomjs": "~1.9.7-9",
-    "browserify": "~4.1.11"
+    "tape": "~2.13.3",
+    "tape-run": "~0.1.1"
   },
   "author": "Tom MacWright <[email protected]>",
   "license": "ISC",