update packages, fix lint warnings (#246)

Author: Sunil Pai

.eslintrc.js

@@ -7,11 +7,8 @@ module.exports = {
     'react',
   ],
 
-  ecmaFeatures: {
-    modules: false
-  },
-
   rules: {
     'no-use-before-define': 2,
+    'max-len': 'off'
   },
 };

.watchmanconfig

@@ -0,0 +1 @@
+{}

package.json

@@ -12,9 +12,9 @@
   },
   "dependencies": {
     "chalk": "^2.4.2",
-    "execa": "^2.0.3",
+    "execa": "^3.2.0",
     "globby": "^10.0.1",
-    "inquirer": "^6.5.0",
+    "inquirer": "^7.0.0",
     "is-git-clean": "^1.1.0",
     "jscodeshift": "^0.6.4",
     "meow": "^5.0.0"
@@ -30,15 +30,15 @@
     ]
   },
   "devDependencies": {
-    "@babel/core": "^7.5.4",
-    "@babel/plugin-proposal-object-rest-spread": "^7.5.4",
-    "@babel/preset-env": "^7.5.4",
-    "babel-eslint": "^6.0.5",
-    "babel-jest": "^24.8.0",
-    "eslint": "^2.13.1",
-    "eslint-plugin-react": "^5.2.2",
+    "@babel/core": "^7.6.4",
+    "@babel/plugin-proposal-object-rest-spread": "^7.6.2",
+    "@babel/preset-env": "^7.6.3",
+    "babel-eslint": "^10.0.3",
+    "babel-jest": "^24.9.0",
+    "eslint": "^6.6.0",
+    "eslint-plugin-react": "^7.16.0",
     "fbjs-scripts": "^0.7.1",
-    "jest": "^24.8.0"
+    "jest": "^24.9.0"
   },
   "bin": {
     "react-codemod": "./bin/react-codemod.js"

transforms/React-DOM-to-react-dom-factories.js

@@ -2,7 +2,7 @@
  * Copyright 2015-present, Facebook, Inc.
  *
  * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree. 
+ * LICENSE file in the root directory of this source tree.
  *
  */
 
@@ -33,26 +33,24 @@ module.exports = function(file, api, options) {
     // } from 'react';
     root
       .find(j.ImportDeclaration)
-      .filter(path => (
-        path.node.specifiers.filter(specifier => (
-          specifier.imported &&
-          specifier.imported.name === DOMModuleName
-        )).length > 0 &&
-        path.node.source.value === 'react'
-      ))
+      .filter(
+        path =>
+          path.node.specifiers.filter(
+            specifier =>
+              specifier.imported && specifier.imported.name === DOMModuleName
+          ).length > 0 && path.node.source.value === 'react'
+      )
       .forEach(path => {
         hasModifications = true;
 
         // Replace the DOM key with 'createElement'
-        path.node.specifiers = path.node.specifiers.map(
-          specifier => {
-            if (specifier.imported && specifier.imported.name === DOMModuleName) {
-              return j.importSpecifier(j.identifier('createElement'));
-            } else {
-              return specifier;
-            }
+        path.node.specifiers = path.node.specifiers.map(specifier => {
+          if (specifier.imported && specifier.imported.name === DOMModuleName) {
+            return j.importSpecifier(j.identifier('createElement'));
+          } else {
+            return specifier;
           }
-        );
+        });
       });
 
     //---------
@@ -69,28 +67,24 @@ module.exports = function(file, api, options) {
     // } = require('react');
     root
       .find(j.ObjectPattern)
-      .filter(path => (
-        path.parent.node.init &&
-        (
+      .filter(
+        path =>
+          path.parent.node.init &&
           // matches '} = React;'
-          path.parent.node.init.name === 'React'
-          ||
-          (
+          (path.parent.node.init.name === 'React' ||
             // matches "} = require('react');"
-            path.parent.node.init.type === 'CallExpression' &&
-            path.parent.node.init.callee.name === 'require' &&
-            path.parent.node.init.arguments[0].value === 'react'
-          )
-        ) &&
-        path.node.properties.some(property => {
-          return property.key.name === DOMModuleName;
-        })
-      ))
+            (path.parent.node.init.type === 'CallExpression' &&
+              path.parent.node.init.callee.name === 'require' &&
+              path.parent.node.init.arguments[0].value === 'react')) &&
+          path.node.properties.some(property => {
+            return property.key.name === DOMModuleName;
+          })
+      )
       .forEach(path => {
         hasModifications = true;
 
         // Replace the DOM key with 'createElement'
-        path.node.properties = path.node.properties.map((property) => {
+        path.node.properties = path.node.properties.map(property => {
           if (property.key.name === DOMModuleName) {
             return j.identifier('createElement');
           } else {
@@ -117,10 +111,9 @@ module.exports = function(file, api, options) {
     function replaceDOMReferences(j, root) {
       let hasModifications = false;
 
-      const isDOMIdentifier = path => (
+      const isDOMIdentifier = path =>
         path.node.name === DOMModuleName &&
-        path.parent.parent.node.type === 'CallExpression'
-      );
+        path.parent.parent.node.type === 'CallExpression';
 
       root
         .find(j.Identifier)
@@ -133,9 +126,7 @@ module.exports = function(file, api, options) {
           const DOMFactoryType = DOMFactoryPath.name;
 
           // DOM.div(... -> createElement(...
-          j(path.parent).replaceWith(
-            j.identifier('createElement')
-          );
+          j(path.parent).replaceWith(j.identifier('createElement'));
           // createElement(... -> createElement('div', ...
           DOMargs.unshift(j.literal(DOMFactoryType));
         });
@@ -155,13 +146,10 @@ module.exports = function(file, api, options) {
     let hasModifications = false;
 
     // matches 'React.DOM'
-    const isReactDOMIdentifier = path => (
+    const isReactDOMIdentifier = path =>
       path.node.name === DOMModuleName &&
-      (
-      path.parent.node.type === 'MemberExpression' &&
-      path.parent.node.object.name === 'React'
-      )
-    );
+      (path.parent.node.type === 'MemberExpression' &&
+        path.parent.node.object.name === 'React');
 
     root
       .find(j.Identifier)
@@ -185,7 +173,5 @@ module.exports = function(file, api, options) {
 
   hasModifications = replaceReactDOMReferences(j, root) || hasModifications;
 
-  return hasModifications
-    ? root.toSource(printOptions)
-    : null;
+  return hasModifications ? root.toSource(printOptions) : null;
 };