Use eslint in the build process.

Author: gajus

.eslintrc

@@ -254,7 +254,7 @@
         "no-array-constructor": 2,
         "no-continue": 2,
         "no-inline-comments": 2,
-        "no-lonely-if": 2,
+        "no-lonely-if": 0,
         "no-mixed-spaces-and-tabs": 2,
         "no-multiple-empty-lines": [
             2,
@@ -382,9 +382,12 @@
         "react"
     ],
     "parser": "babel-eslint",
+    "globals": {
+        "global": true
+    },
     "env": {
-        "browser": true
-        // "mocha": true,
+        "browser": true,
+        "mocha": true
         // "node": true
     }
 }

dist/index.js

@@ -22,8 +22,8 @@ var _makeConfig = require('./makeConfig');
 
 var _makeConfig2 = _interopRequireDefault(_makeConfig);
 
-var functionConstructor = undefined,
-    decoratorConstructor = undefined;
+var decoratorConstructor = undefined,
+    functionConstructor = undefined;
 
 /**
  * When used as a function.

dist/linkClass.js

@@ -22,11 +22,12 @@ linkClass = function (element) {
     var styles = arguments.length <= 1 || arguments[1] === undefined ? {} : arguments[1];
     var options = arguments.length <= 2 || arguments[2] === undefined ? {} : arguments[2];
 
-    var newProps = undefined,
-        newClassName = undefined,
+    var childrenCount = undefined,
+        clonedElement = undefined,
+        moduleName = undefined,
         newChildren = undefined,
-        childrenCount = undefined,
-        moduleName = undefined;
+        newClassName = undefined,
+        newProps = undefined;
 
     if (options.useModuleName) {
         moduleName = element.props.moduleName;
@@ -95,12 +96,12 @@ linkClass = function (element) {
     }
 
     if (newChildren) {
-        element = _react2['default'].cloneElement(element, newProps, newChildren);
+        clonedElement = _react2['default'].cloneElement(element, newProps, newChildren);
     } else {
-        element = _react2['default'].cloneElement(element, newProps);
+        clonedElement = _react2['default'].cloneElement(element, newProps);
     }
 
-    return element;
+    return clonedElement;
 };
 
 exports['default'] = linkClass;

dist/makeConfig.js

@@ -16,6 +16,7 @@ var _utils2 = _interopRequireDefault(_utils);
  * @property {Boolean} keepOriginal
  * @property {Boolean} errorNotFound
  * @property {Boolean} useModuleNam
+ * @return {Object}
  */
 
 exports['default'] = function () {

package.json

@@ -27,14 +27,17 @@
   },
   "devDependencies": {
     "babel": "^5.8.21",
+    "babel-eslint": "^4.1.0",
     "chai": "^3.2.0",
+    "eslint": "^1.2.1",
+    "eslint-plugin-react": "^3.3.0",
     "jsdom": "^6.2.0",
     "mocha": "^2.2.5",
     "react": "^0.14.0-beta3",
     "react-addons-test-utils": "^0.14.0-beta3"
   },
   "scripts": {
-    "test": "mocha",
+    "test": "./node_modules/.bin/eslint ./src/ ./test/ && mocha",
     "build": "babel ./src/ --out-dir ./dist/",
     "watch": "babel --watch ./src/ --out-dir ./dist/"
   }

src/index.js

@@ -1,8 +1,8 @@
 import linkClass from './linkClass';
 import makeConfig from './makeConfig';
 
-let functionConstructor,
-    decoratorConstructor;
+let decoratorConstructor,
+    functionConstructor;
 
 /**
  * When used as a function.

src/linkClass.js

@@ -9,11 +9,12 @@ let linkClass;
  * @return {ReactElement}
  */
 linkClass = (element, styles = {}, options = {}) => {
-    let newProps,
-        newClassName,
+    let childrenCount,
+        clonedElement,
+        moduleName,
         newChildren,
-        childrenCount,
-        moduleName;
+        newClassName,
+        newProps;
 
     if (options.useModuleName) {
         moduleName = element.props.moduleName;
@@ -50,7 +51,7 @@ linkClass = (element, styles = {}, options = {}) => {
             }
         });
 
-        newClassName = newClassName.filter(function (className) {
+        newClassName = newClassName.filter((className) => {
             return className.length;
         });
 
@@ -83,12 +84,12 @@ linkClass = (element, styles = {}, options = {}) => {
     }
 
     if (newChildren) {
-        element = React.cloneElement(element, newProps, newChildren);
+        clonedElement = React.cloneElement(element, newProps, newChildren);
     } else {
-        element = React.cloneElement(element, newProps);
+        clonedElement = React.cloneElement(element, newProps);
     }
 
-    return element;
+    return clonedElement;
 };
 
 export default linkClass;

src/makeConfig.js

@@ -6,6 +6,7 @@ import _ from './utils';
  * @property {Boolean} keepOriginal
  * @property {Boolean} errorNotFound
  * @property {Boolean} useModuleNam
+ * @return {Object}
  */
 export default (userConfig = {}) => {
     let knownProperties,

test/linkClass.js

@@ -169,7 +169,7 @@ describe('linkClass', () => {
     describe('when options.errorNotFound is true', () => {
         it('throws an error when className defines a CSS class that does not exist in CSS modules styles', () => {
             expect(() => {
-                linkClass(<div className='foo'></div>, {}, {errorNotFound: true})
+                linkClass(<div className='foo'></div>, {}, {errorNotFound: true});
             }).to.throw(Error, '"foo" CSS class name is not found in CSS modules styles.');
         });
     });

test/makeConfig.js

@@ -7,6 +7,7 @@ import makeConfig from './../dist/makeConfig';
 describe('makeConfig', () => {
     describe('when using default config', () => {
         let options;
+
         beforeEach(() => {
             options = makeConfig();
         });