[eslint] enable and autofix object-curly-newline and object-property-newline

Author: ljharb

.eslintrc

@@ -30,8 +30,6 @@
         "function-paren-newline": 0,
         "no-plusplus": 1,
         "no-param-reassign": 1,
-        "object-curly-newline": 1,
-        "object-property-newline": 1,
         "no-underscore-dangle": 1,
         "no-mixed-operators": 1,
         "no-void": 1,

lib/rules/jsx-sort-props.js

@@ -118,8 +118,15 @@ const generateFixerFunction = (node, context, reservedList) => {
   // Sort props according to the context. Only supports ignoreCase.
   // Since we cannot safely move JSXSpreadAttribute (due to potential variable overrides),
   // we only consider groups of sortable attributes.
-  const options = {ignoreCase, callbacksLast, shorthandFirst, shorthandLast,
-    noSortAlphabetically, reservedFirst, reservedList};
+  const options = {
+    ignoreCase,
+    callbacksLast,
+    shorthandFirst,
+    shorthandLast,
+    noSortAlphabetically,
+    reservedFirst,
+    reservedList
+  };
   const sortableAttributeGroups = getGroupsOfSortableAttributes(attributes);
   const sortedAttributeGroups = sortableAttributeGroups
     .slice(0)

tests/lib/rules/forbid-elements.js

@@ -105,16 +105,20 @@ ruleTester.run('forbid-elements', rule, {
     },
     {
       code: '<dotted.Component />',
-      options: [{forbid: [
-        {element: 'dotted.Component', message: 'that ain\'t cool'}
-      ]}],
+      options: [{
+        forbid: [
+          {element: 'dotted.Component', message: 'that ain\'t cool'}
+        ]
+      }],
       errors: [{message: '<dotted.Component> is forbidden, that ain\'t cool'}]
     },
     {
       code: '<button />',
-      options: [{forbid: [
-        {element: 'button', message: 'use <Button> instead'}
-      ]}],
+      options: [{
+        forbid: [
+          {element: 'button', message: 'use <Button> instead'}
+        ]
+      }],
       errors: [{message: '<button> is forbidden, use <Button> instead'}]
     },
     {
@@ -143,10 +147,12 @@ ruleTester.run('forbid-elements', rule, {
     },
     {
       code: '<button />',
-      options: [{forbid: [
-        {element: 'button', message: 'use <Button> instead'},
-        {element: 'button', message: 'use <Button2> instead'}
-      ]}],
+      options: [{
+        forbid: [
+          {element: 'button', message: 'use <Button> instead'},
+          {element: 'button', message: 'use <Button2> instead'}
+        ]
+      }],
       errors: [{message: '<button> is forbidden, use <Button2> instead'}]
     },
     {
@@ -164,9 +170,11 @@ ruleTester.run('forbid-elements', rule, {
     },
     {
       code: 'React.createElement(dotted.Component)',
-      options: [{forbid: [
-        {element: 'dotted.Component', message: 'that ain\'t cool'}
-      ]}],
+      options: [{
+        forbid: [
+          {element: 'dotted.Component', message: 'that ain\'t cool'}
+        ]
+      }],
       errors: [{message: '<dotted.Component> is forbidden, that ain\'t cool'}]
     },
     {
@@ -185,9 +193,11 @@ ruleTester.run('forbid-elements', rule, {
     },
     {
       code: 'React.createElement("button")',
-      options: [{forbid: [
-        {element: 'button', message: 'use <Button> instead'}
-      ]}],
+      options: [{
+        forbid: [
+          {element: 'button', message: 'use <Button> instead'}
+        ]
+      }],
       errors: [{message: '<button> is forbidden, use <Button> instead'}]
     },
     {

tests/lib/rules/jsx-boolean-value.js

@@ -55,7 +55,9 @@ ruleTester.run('jsx-boolean-value', rule, {
     output: '<App foo />;',
     errors: [{message: 'Value must be omitted for boolean attributes'}]
   }, {
-    code: '<App foo />;', output: '<App foo={true} />;', options: ['always'],
+    code: '<App foo />;',
+    output: '<App foo={true} />;',
+    options: ['always'],
     errors: [{message: 'Value must be set for boolean attributes'}]
   }, {
     code: '<App foo bar baz />;',

tests/lib/rules/react-in-jsx-scope.js

@@ -44,7 +44,8 @@ ruleTester.run('react-in-jsx-scope', rule, {
     {code: 'var React, App; <App />;'},
     {code: '/** @jsx Foo */ var Foo, App; <App />;'},
     {code: '/** @jsx Foo.Bar */ var Foo, App; <App />;'},
-    {code: `
+    {
+      code: `
       import React from 'react/addons';
       const Button = createReactClass({
         render() {
@@ -54,7 +55,8 @@ ruleTester.run('react-in-jsx-scope', rule, {
         }
       });
       export default Button;
-    `},
+    `
+    },
     {code: 'var Foo, App; <App />;', settings}
   ],
   invalid: [{
@@ -78,6 +80,7 @@ ruleTester.run('react-in-jsx-scope', rule, {
     errors: [{message: '\'Foo\' must be in scope when using JSX'}]
   }, {
     code: 'var React, a = <img />;',
-    errors: [{message: '\'Foo\' must be in scope when using JSX'}], settings
+    errors: [{message: '\'Foo\' must be in scope when using JSX'}],
+    settings
   }]
 });