Merge branch 'develop'

Author: shotamatsuda

.eslintrc

@@ -0,0 +1,3 @@
+{
+  'extends': './index.js',
+}

README.md

@@ -1,7 +1,7 @@
 eslint-config-planck
 ====================
 
-ESLint config for [Planck framework](https://takram.com/projects/planck) and related source codes based on [Airbnb’s](https://github.com/airbnb/javascript/tree/master/packages/eslint-config-airbnb).
+ESLint config for [Planck framework](https://takram.com/projects/planck) and related source codes based on [Airbnb’s](https://github.com/airbnb/javascript).
 
 [![License](http://img.shields.io/badge/license-MIT-lightgrey.svg?style=flat
 )](http://mit-license.org)
@@ -13,15 +13,13 @@ ESLint config for [Planck framework](https://takram.com/projects/planck) and rel
 
 ```sh
 npm install @takram/eslint-config-planck
-
-# Install peer dependencies
 ```
 
 ## License
 
 The MIT License
 
-Copyright (C) 2016-Present Shota Matsuda
+Copyright (C) 2017-Present Shota Matsuda
 
 Permission is hereby granted, free of charge, to any person obtaining a
 copy of this software and associated documentation files (the "Software"),

example.js

@@ -0,0 +1,110 @@
+//
+//  The MIT License
+//
+//  Copyright (C) 2017-Present Shota Matsuda
+//
+//  Permission is hereby granted, free of charge, to any person obtaining a
+//  copy of this software and associated documentation files (the "Software"),
+//  to deal in the Software without restriction, including without limitation
+//  the rights to use, copy, modify, merge, publish, distribute, sublicense,
+//  and/or sell copies of the Software, and to permit persons to whom the
+//  Software is furnished to do so, subject to the following conditions:
+//
+//  The above copyright notice and this permission notice shall be included in
+//  all copies or substantial portions of the Software.
+//
+//  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+//  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+//  FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+//  THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+//  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+//  FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+//  DEALINGS IN THE SOFTWARE.
+//
+
+// arrow-body-style
+{
+  const veryLongNameFunction = () => {
+    return 'this is very long but still a single statement'
+  }
+  veryLongNameFunction()
+}
+
+// arrow-parens
+{
+  const f = a => b => c => {
+    return a + b + c
+  }
+  f(1)(2)(3)
+}
+
+// class-methods-use-this
+{
+  class T {
+    f() {}
+  }
+  new T().f()
+}
+
+// no-bitwise
+{
+  function f(v) {}
+  const v = 1 << 24
+  f(v)
+}
+
+// no-empty
+try {
+  throw new Error()
+} catch (error) {}
+
+// new-cap
+{
+  class T {
+    f() {}
+  }
+  const indirectClass = T
+  const t = new indirectClass()
+  t.f()
+}
+
+// no-mixed-operators
+{
+  function f() {}
+  const v = 1 * 2 + 3
+  f(v)
+}
+
+// no-nested-ternary
+{
+  function f(a, b) {
+    return (a > b) ? 1 : (a < b) ? -1 : 0
+  }
+  f(1, 2)
+}
+
+// no-plusplus
+{
+  function f() {}
+  let i = 0
+  ++i
+  i++
+  f(i)
+}
+
+// no-shadow
+{
+  function f() {}
+  const a = 0
+  {
+    const a = 0
+    f(a)
+  }
+  f(a)
+}
+
+// no-unused-vars
+{
+  function f(a) {}
+  f()
+}

index.js

@@ -22,27 +22,16 @@
 //  DEALINGS IN THE SOFTWARE.
 //
 
-/* eslint-disable quote-props */
-
 module.exports = {
-  'parserOptions': {
-    'ecmaVersion': 2017,
-    'sourceType': 'module',
-    'ecmaFeatures': {
-      'generators': false,
-      'objectLiteralDuplicateProperties': false,
-    },
-  },
-
-  'extends': 'airbnb',
+  extends: 'airbnb',
 
-  'rules': {
+  rules: {
     // Prefer line length limit over this rule
     'arrow-body-style': 0,
 
     // Won't cause any problem
     'arrow-parens': ['error', 'as-needed', {
-      'requireForBlockBody': false,
+      requireForBlockBody: false,
     }],
 
     // Insane
@@ -53,32 +42,45 @@ module.exports = {
 
     // Allow for catch statements
     'no-empty': ['error', {
-      'allowEmptyCatch': true,
+      allowEmptyCatch: true,
     }],
 
     // Limit to 80 except import/export expressions
-    'max-len': ['error', {
-      'code': 80,
-      'ignorePattern': '^\\s*(import|export)',
+    'max-len': ['error', 80, 2, {
+      ignoreUrls: true,
+      ignoreComments: false,
+      ignoreRegExpLiterals: true,
+      ignoreStrings: true,
+      ignoreTemplateLiterals: true,
     }],
 
     // Allow for names ending with "Class"
     'new-cap': ['error', {
-      'newIsCap': true,
-      'newIsCapExceptions': [],
-      'newIsCapExceptionPattern': 'Class$',
-      'capIsNew': false,
+      newIsCap: true,
+      newIsCapExceptions: [],
+      newIsCapExceptionPattern: 'Class$',
+      capIsNew: false,
+      capIsNewExceptions: ['Immutable.Map', 'Immutable.Set', 'Immutable.List'],
     }],
 
     // We know the math
     'no-mixed-operators': ['error', {
-      'groups': [
+      groups: [
+        ['%', '**'],
+        ['%', '+'],
+        ['%', '-'],
+        ['%', '*'],
+        ['%', '/'],
+        ['**', '+'],
+        ['**', '-'],
+        ['**', '*'],
+        ['**', '/'],
         ['&', '|', '^', '~', '<<', '>>', '>>>'],
         ['==', '!=', '===', '!==', '>', '>=', '<', '<='],
         ['&&', '||'],
         ['in', 'instanceof'],
       ],
-      'allowSamePrecedence': false,
+      allowSamePrecedence: false,
     }],
 
     // Not sure and we need to format codes carefully
@@ -92,11 +94,12 @@ module.exports = {
 
     // Insane for arguments
     'no-unused-vars': ['error', {
-      'vars': 'all',
-      'args': 'none',
+      vars: 'all',
+      args: 'none',
+      ignoreRestSiblings: true,
     }],
 
     // Prefer simpler syntax
-    'semi': ['error', 'never'],
+    semi: ['error', 'never'],
   },
 }