Fix array-to-string coercion bug

jsx-eslint · Feb 21, 2018 · 8cc1aa8 · 8cc1aa8
1 parent 0f480f6
commit 8cc1aa8
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/lib/rules/no-typos.js b/lib/rules/no-typos.js
@@ -58,19 +58,14 @@ module.exports = {
     }
 
     function isPropTypesPackage(node) {
-      // Note: we really do want == with the package names here,
-      // since we need value equality, not identity - and
-      // these values are always string or null.
-      /* eslint-disable eqeqeq */
       return (
         node.type === 'Identifier' &&
-        node.name == propTypesPackageName
+        node.name === propTypesPackageName
       ) || (
         node.type === 'MemberExpression' &&
         node.property.name === 'PropTypes' &&
-        node.object.name == reactPackageName
+        node.object.name === reactPackageName
       );
-      /* eslint-enable eqeqeq */
     }
 
     /* eslint-disable no-use-before-define */
@@ -158,7 +153,7 @@ module.exports = {
             node.specifiers
               .filter(specifier => specifier.imported && specifier.imported.name === 'PropTypes')
               .map(specifier => specifier.local.name)
-          ) : null;
+          )[0] : null;
         }
       },