Update formatting, comments, other minor stuff 😎

Author: robertrossmann

coding-styles/react.js

@@ -19,7 +19,9 @@ module.exports = {
 
     // Disallow spaces inside of curly braces in JSX attributes
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-curly-spacing.md
-    'react/jsx-curly-spacing': [1, 'never', { allowMultiline: true }],
+    'react/jsx-curly-spacing': [1, 'never', {
+      allowMultiline: true
+    }],
 
     // Enforce or disallow spaces around equal signs in JSX attributes
     // This rule will enforce consistency of spacing around equal signs in JSX attributes, by

environments/react/best-practices.js

@@ -21,7 +21,9 @@ module.exports = {
 
     // Restrict file extensions that may contain JSX
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-filename-extension.md
-    'react/jsx-filename-extension': [1, { extensions: ['.jsx'] }],
+    'react/jsx-filename-extension': [1, {
+      extensions: ['.jsx']
+    }],
 
     // Detect missing key prop
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-key.md
@@ -50,7 +52,7 @@ module.exports = {
     }],
 
     // Disallow Use of console
-    // Turned off for React apps
+    // Turned off for React apps, console is quite useful in browsers
     'no-console': 0
   }
 }

environments/react/known-errors.js

@@ -17,13 +17,15 @@ module.exports = {
     'react/jsx-no-bind': [2, {
       ignoreRefs: true,
       allowArrowFunctions: true,
-      allowBind: false,
+      allowBind: false
     }],
 
     // Prevent duplicate properties in JSX
     // Creating JSX elements with duplicate props can cause unexpected behavior in your application.
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-no-duplicate-props.md
-    'react/jsx-no-duplicate-props': [2, { ignoreCase: true }],
+    'react/jsx-no-duplicate-props': [2, {
+      ignoreCase: true
+    }],
 
     // Prevent usage of unsafe target='_blank'
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-no-target-blank.md
@@ -79,7 +81,9 @@ module.exports = {
 
     // Prevent multiple component definitions per file
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-multi-comp.md
-    'react/no-multi-comp': [2, { ignoreStateless: true }],
+    'react/no-multi-comp': [2, {
+      ignoreStateless: true
+    }],
 
     // Prevent usage of the return value of React.render
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-render-return-value.md
@@ -100,7 +104,7 @@ module.exports = {
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-unused-prop-types.md
     'react/no-unused-prop-types': [2, {
       customValidators: [],
-      skipShapeProps: false,
+      skipShapeProps: false
     }],
 
     // Enforce ES6 class for React Components
@@ -112,7 +116,10 @@ module.exports = {
     // warn other developers if they make a mistake while reusing the component with improper data
     // type.
     // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/prop-types.md
-    'react/prop-types': [2, { ignore: [], customValidators: [] }],
+    'react/prop-types': [2, {
+      ignore: [],
+      customValidators: []
+    }],
 
     // Prevent missing React when using JSX
     // When using JSX, <a /> expands to React.createElement("a"). Therefore the React variable must

unused.js

@@ -103,37 +103,28 @@ module.exports = {
     // This rule checks all JSX components and verifies that no forbidden propsTypes are used.
     'react/forbid-prop-types': 0,
 
+    // eslint-plugin-react
+
     // Prevent usage of setState
     'react/no-set-state': 0,
 
-    // Limit maximum of props on a single line in JSX
-    'react/max-props-per-line': 0,
-
     // Prevent usage of unwrapped JSX strings
     'react/jsx-no-literals': 0,
 
     // Enforce props alphabetical sorting
-    'react/jsx-sort-props': [0, {
-      ignoreCase: true,
-      callbacksLast: false,
-      shorthandFirst: false,
-      shorthandLast: false,
-    }],
+    'react/jsx-sort-props': 0,
 
     // Limit maximum of props on a single line in JSX
-    'react/jsx-max-props-per-line': [0, { maximum: 1 }],
+    'react/jsx-max-props-per-line': 0,
 
     // Enforce event handler naming conventions in JSX
-    'react/jsx-handler-names': [0, {
-      eventHandlerPrefix: 'handle',
-      eventHandlerPropPrefix: 'on',
-    }],
+    'react/jsx-handler-names': 0,
 
     // Prevent missing displayName in a React component definition
-    'react/display-name': [0, { ignoreTranspilerName: false }],
+    'react/display-name': 0,
 
     // Forbid certain props on Components
-    'react/forbid-component-props': [0, { forbid: [] }],
+    'react/forbid-component-props': 0,
 
     // Enforce React components to have a shouldComponentUpdate method
     'react/require-optimization': 0