Merge branch 'master' into noUsedProps

Author: Diana Suvorova

.eslintrc

@@ -49,6 +49,7 @@
         "guard-for-in": 2,
         "no-alert": 2,
         "no-caller": 2,
+        "no-confusing-arrow": 2,
         "no-div-regex": 2,
         "no-else-return": 2,
         "no-eq-null": 2,
@@ -81,6 +82,7 @@
         "no-void": 0,
         "no-warning-comments": 2,
         "no-with": 2,
+        "prefer-arrow-callback": 2,
         "radix": 2,
         "vars-on-top": 0,
         "wrap-iife": 2,
@@ -105,6 +107,9 @@
         "indent": [2, 2, {
           "SwitchCase": 1
         }],
+        "arrow-body-style": [2, "as-needed"],
+        "arrow-parens": [2, "as-needed"],
+        "arrow-spacing": 2,
         "brace-style": 2,
         "camelcase": 0,
         "comma-spacing": 2,

docs/rules/default-props-match-prop-types.md

@@ -1,4 +1,4 @@
-# Enforce all defaultProps have a corresponding non-required PropType (default-props-match-prop-types)
+# Enforce all defaultProps have a corresponding non-required PropType (react/default-props-match-prop-types)
 
 This rule aims to ensure that any `defaultProp` has a non-required `PropType` declaration.
 
@@ -154,7 +154,7 @@ NotAComponent.propTypes = {
 
 ```js
 ...
-"default-props-match-prop-types": [<enabled>, { "allowRequiredDefaults": <boolean> }]
+"react/default-props-match-prop-types": [<enabled>, { "allowRequiredDefaults": <boolean> }]
 ...
 ```
 

index.js

@@ -90,14 +90,10 @@ function configureAsError(rules) {
   return result;
 }
 
-const activeRules = filterRules(allRules, function(rule) {
-  return !rule.meta.deprecated;
-});
+const activeRules = filterRules(allRules, rule => !rule.meta.deprecated);
 const activeRulesConfig = configureAsError(activeRules);
 
-const deprecatedRules = filterRules(allRules, function(rule) {
-  return rule.meta.deprecated;
-});
+const deprecatedRules = filterRules(allRules, rule => rule.meta.deprecated);
 
 module.exports = {
   deprecatedRules: deprecatedRules,

lib/rules/boolean-prop-naming.js

@@ -40,7 +40,7 @@ module.exports = {
     }]
   },
 
-  create: Components.detect(function(context, components, utils) {
+  create: Components.detect((context, components, utils) => {
     const sourceCode = context.getSourceCode();
     const config = context.options[0] || {};
     const rule = config.rule ? new RegExp(config.rule) : null;
@@ -118,7 +118,7 @@ module.exports = {
       const component = components.get(node) || node;
       const invalidProps = component.invalidProps || [];
 
-      proptypes.forEach(function (prop) {
+      proptypes.forEach(prop => {
         const propKey = getPropKey(prop);
         const flowCheck = (
           prop.type === 'ObjectTypeProperty' &&
@@ -146,7 +146,7 @@ module.exports = {
      * @param {Object} component The component to process
      */
     function reportInvalidNaming(component) {
-      component.invalidProps.forEach(function (propNode) {
+      component.invalidProps.forEach(propNode => {
         const propName = getPropName(propNode);
         context.report({
           node: propNode,
@@ -192,7 +192,7 @@ module.exports = {
         }
 
         // Search for the proptypes declaration
-        node.properties.forEach(function(property) {
+        node.properties.forEach(property => {
           if (!isPropTypesDeclaration(property.key)) {
             return;
           }
@@ -213,7 +213,7 @@ module.exports = {
         }
 
         const list = components.list();
-        Object.keys(list).forEach(function (component) {
+        Object.keys(list).forEach(component => {
           // If this is a functional component that uses a global type, check it
           if (
             list[component].node.type === 'FunctionDeclaration' &&

lib/rules/default-props-match-prop-types.js

@@ -33,7 +33,7 @@ module.exports = {
     }]
   },
 
-  create: Components.detect(function(context, components, utils) {
+  create: Components.detect((context, components, utils) => {
     const configuration = context.options[0] || {};
     const allowRequiredDefaults = configuration.allowRequiredDefaults || false;
     const propWrapperFunctions = new Set(context.settings.propWrapperFunctions || []);
@@ -91,7 +91,7 @@ module.exports = {
      * @returns {ASTNode|null} Return null if the variable could not be found, ASTNode otherwise.
      */
     function findVariableByName(name) {
-      const variable = variableUtil.variablesInScope(context).find((item) => item.name === name);
+      const variable = variableUtil.variablesInScope(context).find(item => item.name === name);
 
       if (!variable || !variable.defs[0] || !variable.defs[0].node) {
         return null;
@@ -139,7 +139,7 @@ module.exports = {
 
     function resolveUnionTypeAnnotation(node) {
       // Go through all the union and resolve any generic types.
-      return node.types.map(function(annotation) {
+      return node.types.map(annotation => {
         if (annotation.type === 'GenericTypeAnnotation') {
           return resolveGenericTypeAnnotation(annotation);
         }
@@ -154,17 +154,13 @@ module.exports = {
      * @returns {Object[]}        Array of PropType object representations, to be consumed by `addPropTypesToComponent`.
      */
     function getPropTypesFromObjectExpression(objectExpression) {
-      const props = objectExpression.properties.filter(function(property) {
-        return property.type !== 'ExperimentalSpreadProperty';
-      });
+      const props = objectExpression.properties.filter(property => property.type !== 'ExperimentalSpreadProperty');
 
-      return props.map(function(property) {
-        return {
-          name: property.key.name,
-          isRequired: isRequiredPropType(property.value),
-          node: property
-        };
-      });
+      return props.map(property => ({
+        name: property.key.name,
+        isRequired: isRequiredPropType(property.value),
+        node: property
+      }));
     }
 
     /**
@@ -188,7 +184,7 @@ module.exports = {
 
         case 'UnionTypeAnnotation':
           const union = resolveUnionTypeAnnotation(node.typeAnnotation);
-          properties = union.reduce(function(acc, curr) {
+          properties = union.reduce((acc, curr) => {
             if (!curr) {
               return acc;
             }
@@ -206,11 +202,9 @@ module.exports = {
           break;
       }
 
-      const props = properties.filter(function(property) {
-        return property.type === 'ObjectTypeProperty';
-      });
+      const props = properties.filter(property => property.type === 'ObjectTypeProperty');
 
-      return props.map(function(property) {
+      return props.map(property => {
         // the `key` property is not present in ObjectTypeProperty nodes, so we need to get the key name manually.
         const tokens = context.getFirstTokens(property, 1);
         const name = tokens[0].value;
@@ -237,12 +231,10 @@ module.exports = {
         return 'unresolved';
       }
 
-      return objectExpression.properties.map(function(defaultProp) {
-        return {
-          name: defaultProp.key.name,
-          node: defaultProp
-        };
-      });
+      return objectExpression.properties.map(defaultProp => ({
+        name: defaultProp.key.name,
+        node: defaultProp
+      }));
     }
 
     /**
@@ -348,7 +340,7 @@ module.exports = {
         return;
       }
 
-      defaultProps.forEach(function(defaultProp) {
+      defaultProps.forEach(defaultProp => {
         const prop = propFromName(propTypes, defaultProp.name);
 
         if (prop && (allowRequiredDefaults || !prop.isRequired)) {
@@ -567,7 +559,7 @@ module.exports = {
         }
 
         // Search for the proptypes declaration
-        node.properties.forEach(function(property) {
+        node.properties.forEach(property => {
           if (property.type === 'ExperimentalSpreadProperty') {
             return;
           }

lib/rules/display-name.js

@@ -30,7 +30,7 @@ module.exports = {
     }]
   },
 
-  create: Components.detect(function(context, components, utils) {
+  create: Components.detect((context, components, utils) => {
     const sourceCode = context.getSourceCode();
     const config = context.options[0] || {};
     const ignoreTranspilerName = config.ignoreTranspilerName || false;
@@ -209,7 +209,7 @@ module.exports = {
       ObjectExpression: function(node) {
         if (ignoreTranspilerName || !hasTranspilerName(node)) {
           // Search for the displayName declaration
-          node.properties.forEach(function(property) {
+          node.properties.forEach(property => {
             if (!property.key || !isDisplayNameDeclaration(property.key)) {
               return;
             }

lib/rules/forbid-elements.js

@@ -50,7 +50,7 @@ module.exports = {
 
     const indexedForbidConfigs = {};
 
-    forbidConfiguration.forEach(function(item) {
+    forbidConfiguration.forEach(item => {
       if (typeof item === 'string') {
         indexedForbidConfigs[item] = {element: item};
       } else {

lib/rules/forbid-foreign-prop-types.js

@@ -45,9 +45,7 @@ module.exports = {
       },
 
       ObjectPattern: function(node) {
-        const propTypesNode = node.properties.find(function(property) {
-          return property.type === 'Property' && property.key.name === 'propTypes';
-        });
+        const propTypesNode = node.properties.find(property => property.type === 'Property' && property.key.name === 'propTypes');
 
         if (propTypesNode) {
           context.report({

lib/rules/forbid-prop-types.js

@@ -74,7 +74,7 @@ module.exports = {
      * @returns {void}
      */
     function checkForbidden(declarations) {
-      declarations.forEach(function(declaration) {
+      declarations.forEach(declaration => {
         if (declaration.type !== 'Property') {
           return;
         }
@@ -154,7 +154,7 @@ module.exports = {
       },
 
       ObjectExpression: function(node) {
-        node.properties.forEach(function(property) {
+        node.properties.forEach(property => {
           if (!property.key) {
             return;
           }

lib/rules/jsx-boolean-value.js

@@ -20,7 +20,7 @@ const NEVER = 'never';
 const errorData = new WeakMap();
 function getErrorData(exceptions) {
   if (!errorData.has(exceptions)) {
-    const exceptionProps = Array.from(exceptions, (name) => `\`${name}\``).join(', ');
+    const exceptionProps = Array.from(exceptions, name => `\`${name}\``).join(', ');
     const exceptionsMessage = exceptions.size > 0 ? ` for the following props: ${exceptionProps}` : '';
     errorData.set(exceptions, {exceptionsMessage: exceptionsMessage});
   }