Meeting the updated linting requirements.

Author: gajus

.babelrc

@@ -2,7 +2,6 @@
 
 [![Travis build status](http://img.shields.io/travis/gajus/react-css-modules/master.svg?style=flat)](https://travis-ci.org/gajus/react-css-modules)
 [![NPM version](http://img.shields.io/npm/v/react-css-modules.svg?style=flat)](https://www.npmjs.org/package/react-css-modules)
-[![Coverage Status](https://coveralls.io/repos/gajus/react-css-modules/badge.svg?branch=master&service=github)](https://coveralls.io/github/gajus/react-css-modules?branch=master)
 
 <img src='./README/react-css-modules.png' height='150' />
 

README.md

@@ -1,7 +1,7 @@
 {
   "name": "react-css-modules",
   "description": "Seamless mapping of class names to CSS modules inside of React components.",
-  "main": "./dist/index.js",
+  "main": "./dist/",
   "repository": {
     "type": "git",
     "url": "https://github.com/gajus/react-css-modules"
@@ -15,25 +15,28 @@
   "version": "3.6.1",
   "author": {
     "name": "Gajus Kuizinas",
-    "email": "gk@anuary.com",
+    "email": "gajus@gajus.com",
     "url": "http://gajus.com"
   },
   "license": "BSD-3-Clause",
   "dependencies": {
     "lodash": "^3.10.1"
   },
   "devDependencies": {
-    "chai": "^3.4.0",
-    "coveralls": "^2.11.4",
-    "jsdom": "^7.0.2",
-    "pragmatist": "^1.1.5",
-    "react": "^0.14.0",
-    "react-addons-test-utils": "^0.14.0"
+    "chai": "^3.4.1",
+    "jsdom": "^7.1.1",
+    "pragmatist": "^1.9.4",
+    "react": "^0.14.3",
+    "react-addons-test-utils": "^0.14.3"
   },
   "scripts": {
     "pragmatist": "node ./node_modules/.bin/pragmatist",
     "lint": "npm run pragmatist lint",
+    "test": "npm run pragmatist test",
     "build": "npm run pragmatist build",
-    "test": "npm run pragmatist test && cat ./coverage/lcov.info | ./node_modules/.bin/coveralls"
+    "watch": "npm run pragmatist watch",
+    "watch-lint": "npm run pragmatist watch-lint",
+    "watch-test": "npm run pragmatist watch-test",
+    "watch-build": "npm run pragmatist watch-build"
   }
 }

package.json

@@ -1,3 +1,5 @@
+/* eslint-disable react/prop-types */
+
 import linkClass from './linkClass';
 import React from 'react';
 import _ from 'lodash';

src/extendReactClass.js

@@ -1,3 +1,4 @@
+import _ from 'lodash';
 import extendReactClass from './extendReactClass';
 import wrapStatelessFunction from './wrapStatelessFunction';
 
@@ -12,7 +13,7 @@ let decoratorConstructor,
  * @return {boolean}
  */
 isReactComponent = (Component) => {
-    return 'prototype' in Component && typeof Component.prototype.render === 'function';
+    return 'prototype' in Component && _.isFunction(Component.prototype.render);
 };
 
 /**
@@ -55,7 +56,7 @@ decoratorConstructor = (defaultStyles, options) => {
 };
 
 export default (...args) => {
-    if (typeof args[0] === 'function') {
+    if (_.isFunction(args[0])) {
         return functionConstructor(args[0], args[1], args[2]);
     } else {
         return decoratorConstructor(args[0], args[1]);

src/index.js

@@ -1,10 +1,29 @@
-import { isObject } from 'lodash';
+import _ from 'lodash';
 
-const ITERATOR_SYMBOL = typeof Symbol === 'function' && Symbol.iterator;
-const OLD_ITERATOR_SYMBOL = '@@iterator';
+let ITERATOR_SYMBOL,
+    OLD_ITERATOR_SYMBOL;
 
-export default function isIterable(obj) {
-  return isObject(obj) &&
-    typeof ((ITERATOR_SYMBOL && obj[ITERATOR_SYMBOL])
-        || obj[OLD_ITERATOR_SYMBOL]) === 'function';
-}
+ITERATOR_SYMBOL = _.isFunction(Symbol) && Symbol.iterator;
+OLD_ITERATOR_SYMBOL = '@@iterator';
+
+/**
+ * @see https://github.com/lodash/lodash/issues/1668
+ * @see https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Iteration_protocols
+ * @param {Object} target
+ * @returns {boolean}
+ */
+export default (target) => {
+    let iterator;
+
+    if (!_.isObject(target)) {
+        return false;
+    }
+
+    if (ITERATOR_SYMBOL) {
+        iterator = target[ITERATOR_SYMBOL];
+    } else {
+        iterator = target[OLD_ITERATOR_SYMBOL];
+    }
+
+    return _.isFunction(iterator);
+};

src/isIterable.js

@@ -34,24 +34,22 @@ linkClass = (element, styles = {}, userConfiguration) => {
         styleNames = _.filter(styleNames);
 
         if (configuration.allowMultiple === false && styleNames.length > 1) {
-            throw new Error(`ReactElement styleName property defines multiple module names ("${element.props.styleName}").`);
+            throw new Error('ReactElement styleName property defines multiple module names ("' + element.props.styleName + '").');
         }
 
-        appendClassName = styleNames.map((styleName) => {
+        appendClassName = _.map(styleNames, (styleName) => {
             if (styles[styleName]) {
                 return styles[styleName];
             } else {
                 if (configuration.errorWhenNotFound === true) {
-                    throw new Error(`"${styleName}" CSS module is undefined.`);
+                    throw new Error('"' + styleName + '" CSS module is undefined.');
                 }
 
                 return '';
             }
         });
 
-        appendClassName = appendClassName.filter((className) => {
-            return className.length;
-        });
+        appendClassName = _.filter(appendClassName, 'length');
 
         appendClassName = appendClassName.join(' ');
     }
@@ -62,14 +60,14 @@ linkClass = (element, styles = {}, userConfiguration) => {
     // [ReactElement, 'text']
     // ReactElement
 
-    // console.log(`element.props.children`, element.props.children, `React.Children.count(element.props.children)`, React.Children.count(element.props.children));
-
     children = element.props.children;
 
     if (React.isValidElement(children)) {
         newChildren = linkClass(React.Children.only(children), styles, configuration);
     } else if (_.isArray(children) || isIterable(children)) {
+        /* eslint-disable lodash3/prefer-lodash-method */
         newChildren = React.Children.map(children, (node) => {
+        /* eslint-enable lodash3/prefer-lodash-method */
             if (React.isValidElement(node)) {
                 return linkClass(node, styles, configuration);
             } else {
@@ -85,7 +83,7 @@ linkClass = (element, styles = {}, userConfiguration) => {
 
     if (appendClassName) {
         if (element.props.className) {
-            appendClassName = `${element.props.className} ${appendClassName}`;
+            appendClassName = element.props.className + ' ' + appendClassName;
         }
 
         newProps = {

src/linkClass.js

@@ -20,12 +20,12 @@ export default (userConfiguration = {}) => {
     };
 
     _.forEach(userConfiguration, (value, name) => {
-        if (typeof configuration[name] === 'undefined') {
-            throw new Error(`Unknown configuration property "${name}".`);
+        if (_.isUndefined(configuration[name])) {
+            throw new Error('Unknown configuration property "' + name + '".');
         }
 
-        if (typeof value !== 'boolean') {
-            throw new Error(`"${name}" property value must be a boolean.`);
+        if (!_.isBoolean(value)) {
+            throw new Error('"' + name + '" property value must be a boolean.');
         }
 
         configuration[name] = value;

src/makeConfiguration.js

@@ -1,3 +1,5 @@
+/* eslint-disable react/prop-types */
+
 import linkClass from './linkClass';
 import React from 'react';
 import _ from 'lodash';

src/wrapStatelessFunction.js

@@ -11,15 +11,7 @@ import extendReactClass from './../src/extendReactClass';
 
 describe('extendReactClass', () => {
     beforeEach(() => {
-        global.document = jsdom.jsdom(`
-            <!DOCTYPE html>
-            <html>
-            <head>
-            </head>
-            <body>
-            </body>
-            </html>
-        `);
+        global.document = jsdom.jsdom('<!DOCTYPE html><html><head></head><body></body></html>');
 
         global.window = document.defaultView;
     });