remove remaining style-only rules, add additional correctness rules, …

…update deps (#8)

* remove remaining style-only rules
* adds additional correctness rules
* updates dependencies

new projects will use this eslint config, and we want it to just be standardjs plus additional rules that help enforce correctness, no style. each existing project can add its own style rules or (suggested) use eslint --fix to automatically adhere to this config.

reviewers: @samskeller @sawyerh

README.md

@@ -4,9 +4,9 @@
 
 We're based on [Standard JS](http://standardjs.com/), which provides a complete node/browser/react ruleset, with a few changes:
 
-* using `eslint` directly, since we want to relax rulesets for adoption into existing repos
-* enforcing use of semicolons
-* locking to es6, as we don't assume babel compilation for node services
+* using `eslint-config-standard` instead of `standard`, since we want to add additional rules
+* adding additional rules that either promote correctness or a higher signal-to-noise ratio
+* avoiding rules that are purely stylistic in nature
 
 [See the full Standard JS ruleset here](https://github.com/feross/eslint-config-standard/blob/master/eslintrc.json) and the [Standard React ruleset here](https://github.com/feross/eslint-config-standard-react/blob/master/eslintrc.json).
 

index.js

@@ -1,20 +1,16 @@
 module.exports = {
   extends: ['standard', 'standard-react'],
   rules: {
-    /*
-      Standard overrides
-    */
-    semi: ['error', 'always'], // churn-expensive to enforce
-    'space-before-function-paren': ['error', 'never'],
-
     /*
       Nava-specific rules
     */
-    'linebreak-style': ['error', 'unix'],
-    'sort-vars': ['error', {ignoreCase: true}],
     'no-alert': 'error',
     'no-var': 'error',
     'prefer-const': 'error',
-    'prefer-arrow-callback': 'error'
+    'prefer-arrow-callback': 'error',
+    'promise/no-native': 'error',
+    'promise/catch-or-return': 'error',
+    'promise/no-return-wrap': 'error',
+    'promise/valid-params': 'error'
   }
 };