[eslint config] [*] fix quoting of rule properties.

Per airbnb#885.

Author: ljharb

packages/eslint-config-airbnb-base/rules/.eslintrc.json

@@ -1,5 +1,5 @@
 module.exports = {
-  'rules': {
+  rules: {
     // enforces getter/setter pairs in objects
     'accessor-pairs': 0,
 
@@ -11,26 +11,26 @@ module.exports = {
     'block-scoped-var': 2,
 
     // specify the maximum cyclomatic complexity allowed in a program
-    'complexity': [0, 11],
+    complexity: [0, 11],
 
     // require return statements to either always or never specify values
     'consistent-return': 2,
 
     // specify curly brace conventions for all control statements
-    'curly': [2, 'multi-line'],
+    curly: [2, 'multi-line'],
 
     // require default case in switch statements
-    'default-case': [2, { 'commentPattern': '^no default$' }],
+    'default-case': [2, { commentPattern: '^no default$' }],
 
     // encourages use of dot notation whenever possible
-    'dot-notation': [2, { 'allowKeywords': true }],
+    'dot-notation': [2, { allowKeywords: true }],
 
     // enforces consistent newlines before or after dots
     'dot-location': 0,
 
     // require the use of === and !==
     // http://eslint.org/docs/rules/eqeqeq
-    'eqeqeq': [2, 'allow-null'],
+    eqeqeq: [2, 'allow-null'],
 
     // make sure for-in loops have an if statement
     'guard-for-in': 2,
@@ -54,7 +54,7 @@ module.exports = {
     // disallow empty functions, except for standalone funcs/arrows
     // http://eslint.org/docs/rules/no-empty-function
     'no-empty-function': [2, {
-      'allow': [
+      allow: [
         'arrowFunctions',
         'functions',
         'methods',
@@ -104,7 +104,7 @@ module.exports = {
     'no-iterator': 2,
 
     // disallow use of labels for anything other then loops and switches
-    'no-labels': [2, { 'allowLoop': false, 'allowSwitch': false }],
+    'no-labels': [2, { allowLoop: false, allowSwitch: false }],
 
     // disallow unnecessary nested blocks
     'no-lone-blocks': 2,
@@ -115,10 +115,10 @@ module.exports = {
     // disallow magic numbers
     // http://eslint.org/docs/rules/no-magic-numbers
     'no-magic-numbers': [0, {
-      'ignore': [],
-      'ignoreArrayIndexes': true,
-      'enforceConst': true,
-      'detectObjects': false,
+      ignore: [],
+      ignoreArrayIndexes: true,
+      enforceConst: true,
+      detectObjects: false,
     }],
 
     // disallow use of multiple spaces
@@ -149,7 +149,7 @@ module.exports = {
     // disallow reassignment of function parameters
     // disallow parameter object manipulation
     // rule: http://eslint.org/docs/rules/no-param-reassign.html
-    'no-param-reassign': [2, { 'props': true }],
+    'no-param-reassign': [2, { props: true }],
 
     // disallow usage of __proto__ property
     'no-proto': 2,
@@ -202,13 +202,13 @@ module.exports = {
     'no-void': 0,
 
     // disallow usage of configurable warning terms in comments: e.g. todo
-    'no-warning-comments': [0, { 'terms': ['todo', 'fixme', 'xxx'], 'location': 'start' }],
+    'no-warning-comments': [0, { terms: ['todo', 'fixme', 'xxx'], location: 'start' }],
 
     // disallow use of the with statement
     'no-with': 2,
 
     // require use of the second argument for parseInt()
-    'radix': 2,
+    radix: 2,
 
     // requires to declare all vars on top of their containing scope
     'vars-on-top': 2,
@@ -218,6 +218,6 @@ module.exports = {
     'wrap-iife': [2, 'outside'],
 
     // require or disallow Yoda conditions
-    'yoda': 2
+    yoda: 2
   }
 };

packages/eslint-config-airbnb-base/rules/best-practices.js

@@ -1,5 +1,5 @@
 module.exports = {
-  'rules': {
+  rules: {
     // require trailing commas in multiline object literals
     'comma-dangle': [2, 'always-multiline'],
 
@@ -42,9 +42,9 @@ module.exports = {
     // disallow unnecessary parentheses
     // http://eslint.org/docs/rules/no-extra-parens
     'no-extra-parens': [0, 'all', {
-      'conditionalAssign': true,
-      'nestedBinaryExpressions': false,
-      'returnAssign': false,
+      conditionalAssign: true,
+      nestedBinaryExpressions: false,
+      returnAssign: false,
     }],
 
     // disallow unnecessary semicolons

packages/eslint-config-airbnb-base/rules/errors.js

@@ -1,17 +1,17 @@
 module.exports = {
-  'env': {
-    'es6': true
+  env: {
+    es6: true
   },
-  'parserOptions': {
-    'ecmaVersion': 6,
-    'sourceType': 'module',
-    'ecmaFeatures': {
-      'generators': false,
-      'objectLiteralDuplicateProperties': false
+  parserOptions: {
+    ecmaVersion: 6,
+    sourceType: 'module',
+    ecmaFeatures: {
+      generators: false,
+      objectLiteralDuplicateProperties: false
     }
   },
 
-  'rules': {
+  rules: {
     // enforces no braces where they can be omitted
     // http://eslint.org/docs/rules/arrow-body-style
     'arrow-body-style': [2, 'as-needed'],
@@ -21,14 +21,14 @@ module.exports = {
 
     // require space before/after arrow function's arrow
     // http://eslint.org/docs/rules/arrow-spacing
-    'arrow-spacing': [2, { 'before': true, 'after': true }],
+    'arrow-spacing': [2, { before: true, after: true }],
 
     // verify super() callings in constructors
     'constructor-super': 0,
 
     // enforce the spacing around the * in generator functions
     // http://eslint.org/docs/rules/generator-star-spacing
-    'generator-star-spacing': [2, { 'before': false, 'after': true }],
+    'generator-star-spacing': [2, { before: false, after: true }],
 
     // disallow modifying variables of class declarations
     // http://eslint.org/docs/rules/no-class-assign
@@ -37,7 +37,7 @@ module.exports = {
     // disallow arrow functions where they could be confused with comparisons
     // http://eslint.org/docs/rules/no-confusing-arrow
     'no-confusing-arrow': [2, {
-      'allowParens': true,
+      allowParens: true,
     }],
 
     // disallow modifying variables that are declared using const
@@ -76,20 +76,20 @@ module.exports = {
     // require method and property shorthand syntax for object literals
     // http://eslint.org/docs/rules/object-shorthand
     'object-shorthand': [2, 'always', {
-      'ignoreConstructors': false,
-      'avoidQuotes': true,
+      ignoreConstructors: false,
+      avoidQuotes: true,
     }],
 
     // suggest using arrow functions as callbacks
     'prefer-arrow-callback': [2, {
-      'allowNamedFunctions': false,
-      'allowUnboundThis': true,
+      allowNamedFunctions: false,
+      allowUnboundThis: true,
     }],
 
     // suggest using of const declaration for variables that are never modified after declared
     'prefer-const': [2, {
-      'destructuring': 'any',
-      'ignoreReadBeforeAssign': true,
+      destructuring: 'any',
+      ignoreReadBeforeAssign: true,
     }],
 
     // suggest using Reflect methods where applicable

packages/eslint-config-airbnb-base/rules/es6.js

@@ -1,29 +1,29 @@
 module.exports = {
-  'env': {
-    'es6': true
+  env: {
+    es6: true
   },
-  'parserOptions': {
-    'ecmaVersion': 6,
-    'sourceType': 'module'
+  parserOptions: {
+    ecmaVersion: 6,
+    sourceType: 'module'
   },
-  'plugins': [
+  plugins: [
     'import'
   ],
 
-  'settings': {
+  settings: {
     'import/resolver': {
-      'node': {
-        'extensions': ['.js', '.json']
+      node: {
+        extensions: ['.js', '.json']
       }
     }
   },
 
-  'rules': {
+  rules: {
     // Static analysis:
 
     // ensure imports point to files/modules that can be resolved
     // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-unresolved.md
-    'import/no-unresolved': [2, { 'commonjs': true }],
+    'import/no-unresolved': [2, { commonjs: true }],
 
     // ensure named imports coupled with named exports
     // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/named.md#when-not-to-use-it
@@ -60,8 +60,8 @@ module.exports = {
     // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-extraneous-dependencies.md
     // TODO: enable
     'import/no-extraneous-dependencies': [0, {
-      'devDependencies': false,
-      'optionalDependencies': false,
+      devDependencies: false,
+      optionalDependencies: false,
     }],
 
     // Forbid mutable exports
@@ -106,7 +106,7 @@ module.exports = {
     // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/order.md
     // TODO: enable?
     'import/order': [0, {
-      'groups': ['builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
+      groups: ['builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
       'newlines-between': 'never',
     }],
 

packages/eslint-config-airbnb-base/rules/imports.js

@@ -1,9 +1,9 @@
 module.exports = {
-  'env': {
-    'node': true
+  env: {
+    node: true
   },
 
-  'rules': {
+  rules: {
     // enforce return after a callback
     'callback-return': 0,
 

packages/eslint-config-airbnb-base/rules/node.js

@@ -1,6 +1,6 @@
 module.exports = {
-  'rules': {
+  rules: {
     // babel inserts `'use strict';` for us
-    'strict': [2, 'never']
+    strict: [2, 'never']
   }
 };