Merge pull request #16 from shakacode/13.0.1-remove-requirement-for-a…

…rrow-parens 13.0.1 remove requirement for arrow parens
shakacode · Nov 14, 2016 · 41c571e · 41c571e
2 parents f303472 + e8e652e
commit 41c571e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/packages/eslint-config-shakacode/package.json b/packages/eslint-config-shakacode/package.json
@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-shakacode",
-  "version": "13.0.0",
+  "version": "13.0.1-beta.1",
   "description": "Shakacode eslint config.",
   "authors": [
     "Justin Gordon <[email protected]> (https://github.com/justin808)",

diff --git a/packages/eslint-config-shakacode/rules/javascript.yml b/packages/eslint-config-shakacode/rules/javascript.yml
@@ -1,8 +1,11 @@
 ---
 rules:
+  # Airbnb style guideline has arrow-parens to be used if the body has braces, but not parens.
+  # We use flow, so we need the parens sometimes for single params with type annotations.
+  # However, we can't always use parens, as we don't want them for simple lodash operations with
+  # inline arrow functions.
   arrow-parens:
-    - 2
-    - "always"
+    - 0
 
   # http://eslint.org/docs/rules/no-return-assign.html
   # packages/eslint-config-airbnb-base/rules/best-practices.js used the default which is