Merge pull request #58 from keyanzhang/better-linter

Upgrade eslint and jest

Author: cpojer

.eslintrc

@@ -0,0 +1,13 @@
+module.exports = {
+  parser: 'babel-eslint',
+
+  extends: './node_modules/fbjs-scripts/eslint/.eslintrc.js',
+
+  plugins: [
+    'react',
+  ],
+
+  ecmaFeatures: {
+    modules: false
+  },
+};

.eslintrc.js

@@ -9,25 +9,26 @@
     "lint": "eslint ."
   },
   "dependencies": {
-    "jscodeshift": "^0.3.20",
-    "babel-eslint": "^5.0.0",
+    "jscodeshift": "^0.3.23",
+    "babel-eslint": "^6.0.5",
     "babel-plugin-transform-object-rest-spread": "^6.6.5",
     "babel-preset-es2015": "^6.6.0",
-    "babel-jest": "^9.0.2",
-    "eslint": "^1.7.3",
-    "fbjs-scripts": "^0.5.0",
-    "jest-cli": "^0.9.1"
+    "babel-jest": "^12.1.0",
+    "eslint": "^2.13.1",
+    "fbjs-scripts": "^0.7.1",
+    "jest-cli": "^12.1.0"
   },
   "jest": {
     "automock": false,
     "globals": {
       "baseDir": "../"
     },
+    "testEnvironment": "node",
     "testPathDirs": [
-      "transforms/__tests__"
+      "transforms"
     ]
   },
   "devDependencies": {
-    "eslint-plugin-react": "^5.1.1"
+    "eslint-plugin-react": "^5.2.2"
   }
 }

package.json

@@ -2,3 +2,4 @@
 rules:
   no-undef: 0
   no-unused-vars: 0
+  no-redeclare: 0

transforms/__testfixtures__/.eslintrc

@@ -76,6 +76,7 @@ var MyComponent3 = React.createClass({
         />
       );
     }
+    return null;
   },
 
   autobindMe: function() {},

transforms/__testfixtures__/class.input.js

@@ -66,6 +66,7 @@ class MyComponent3 extends React.Component {
         />
       );
     }
+    return null;
   }
 
   autobindMe() {}

transforms/__testfixtures__/class.output.js

@@ -72,6 +72,7 @@ module.exports = function(file, api, options) {
             value
           );
         }
+        return null;
       });
 
       return attributes;

transforms/create-element-to-jsx.js

@@ -169,6 +169,7 @@ module.exports = function(j) {
         return spec;
       }
     }
+    return null;
   };
 
   const getClassExtendReactSpec = classPath => classPath.value.body;