Merge updates from project template

Signed-off-by: Kevin Locke <[email protected]>

Author: kevinoid

.eslintrc.yaml

@@ -8,12 +8,6 @@ extends: "airbnb-base"
 parserOptions:
   sourceType: "script"
 
-env:
-  node: true
-  ## disable unwanted browser env from airbnb/legacy
-  # Note: Only works for ESLint 2+: https://github.com/eslint/eslint/issues/3915
-  browser: false
-
 rules:
   # Possible Errors
   # list: https://github.com/eslint/eslint/tree/master/docs/rules#possible-errors
@@ -31,8 +25,6 @@ rules:
   no-param-reassign: 0
   ## disallow unnecessary use of Function.prototype.{apply,call}
   no-useless-call: 2
-  ## allow vars anywhere that satisfies other linting constraints
-  vars-on-top: 0
 
   # Strict Mode
   # list: https://github.com/eslint/eslint/tree/master/docs/rules#strict-mode
@@ -91,7 +83,8 @@ rules:
   one-var-declaration-per-line: [2, "initializations"]
   ## Requires operator at the beginning of the line in multiline statements
   # Airbnb prevents breaks around =, suggesting ().  I don't see the advantage.
-  operator-linebreak: [2, "before"]
+  # Break after = looks better to me, so first assigned operand is farther left.
+  operator-linebreak: [2, "before", { "overrides": { "=": "after" } }]
   ## no space before function, eg. 'function()'
   space-before-function-paren: [2, "never"]
 

.githooks/pre-commit/01-nonascii-and-whitespace.sh

@@ -0,0 +1,47 @@
+#!/bin/sh
+#
+# Enforce hooks.allownonascii and core.whitespace configuration on commit.
+#
+# Copied from:
+# https://github.com/git/git/blob/v2.18.0/templates/hooks--pre-commit.sample
+
+if git rev-parse --verify HEAD >/dev/null 2>&1
+then
+	against=HEAD
+else
+	# Initial commit: diff against an empty tree object
+	against=$(git hash-object -t tree /dev/null)
+fi
+
+# If you want to allow non-ASCII filenames set this variable to true.
+allownonascii=$(git config --bool hooks.allownonascii)
+
+# Redirect output to stderr.
+exec 1>&2
+
+# Cross platform projects tend to avoid non-ASCII filenames; prevent
+# them from being added to the repository. We exploit the fact that the
+# printable range starts at the space character and ends with tilde.
+if [ "$allownonascii" != "true" ] &&
+	# Note that the use of brackets around a tr range is ok here, (it's
+	# even required, for portability to Solaris 10's /usr/bin/tr), since
+	# the square bracket bytes happen to fall in the designated range.
+	test $(git diff --cached --name-only --diff-filter=A -z $against |
+	  LC_ALL=C tr -d '[ -~]\0' | wc -c) != 0
+then
+	cat <<\EOF
+Error: Attempt to add a non-ASCII file name.
+
+This can cause problems if you want to work with people on other platforms.
+
+To be portable it is advisable to rename the file.
+
+If you know what you are doing you can disable this check using:
+
+  git config hooks.allownonascii true
+EOF
+	exit 1
+fi
+
+# If there are whitespace errors, print the offending file names and fail.
+exec git diff-index --check --cached $against --

.githooks/pre-commit/02-eslint.sh

@@ -0,0 +1,13 @@
+#!/bin/sh
+# Run ESLint on changed .js and .jsx files
+
+set -Ceu
+
+# Run ESLint on added/modified JS files
+# Use -z to \0-separate and prevent quoting of special chars in filenames
+# Use tr+grep+tr to filter newline separated (could use --null-data for GNU)
+git diff -z --cached --name-only --diff-filter=AM |
+	tr '\n\0' '\0\n' |
+	grep -a '\.jsx\?$' |
+	tr '\0\n' '\n\0' |
+	xargs -0r ./node_modules/.bin/eslint --

package.json

@@ -62,11 +62,6 @@
     "node": ">=6",
     "npm": ">=1.3.7"
   },
-  "greenkeeper": {
-    "ignore": [
-      "eslint"
-    ]
-  },
   "nyc": {
     "exclude": [
       "test",