Add build process for createReactClass

.eslintrc.js

@@ -21,7 +21,7 @@ module.exports = {
     'dot-notation': ERROR,
     'eol-last': ERROR,
     'eqeqeq': [ERROR, 'allow-null'],
-    'indent': [ERROR, 2, {SwitchCase: 1}],
+    'indent': OFF, // We use Prettier now
     'jsx-quotes': [ERROR, 'prefer-double'],
     'keyword-spacing': [ERROR, {after: true, before: true}],
     'no-bitwise': OFF,

addons/create-react-class/.gitignore

@@ -0,0 +1,2 @@
+create-react-class.js
+create-react-class.min.js

addons/create-react-class/factory.js

@@ -314,7 +314,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
       Constructor.childContextTypes = _assign(
         {},
         Constructor.childContextTypes,
-        childContextTypes,
+        childContextTypes
       );
     },
     contextTypes: function(Constructor, contextTypes) {
@@ -324,7 +324,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
       Constructor.contextTypes = _assign(
         {},
         Constructor.contextTypes,
-        contextTypes,
+        contextTypes
       );
     },
     /**
@@ -335,7 +335,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
       if (Constructor.getDefaultProps) {
         Constructor.getDefaultProps = createMergedResultFunction(
           Constructor.getDefaultProps,
-          getDefaultProps,
+          getDefaultProps
         );
       } else {
         Constructor.getDefaultProps = getDefaultProps;
@@ -365,7 +365,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
               'React.PropTypes.',
             Constructor.displayName || 'ReactClass',
             ReactPropTypeLocationNames[location],
-            propName,
+            propName
           );
         }
       }
@@ -384,7 +384,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
         'ReactClassInterface: You are attempting to override ' +
           '`%s` from your class specification. Ensure that your method names ' +
           'do not overlap with React methods.',
-        name,
+        name
       );
     }
 
@@ -395,7 +395,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
         'ReactClassInterface: You are attempting to define ' +
           '`%s` on your component more than once. This conflict may be due ' +
           'to a mixin.',
-        name,
+        name
       );
     }
   }
@@ -418,7 +418,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
               'as well as any mixins they include themselves. ' +
               'Expected object but got %s.',
             Constructor.displayName || 'ReactClass',
-            spec === null ? null : typeofSpec,
+            spec === null ? null : typeofSpec
           );
         }
       }
@@ -430,12 +430,12 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
       typeof spec !== 'function',
       "ReactClass: You're attempting to " +
         'use a component class or function as a mixin. Instead, just use a ' +
-        'regular object.',
+        'regular object.'
     );
     _invariant(
       !isValidElement(spec),
       "ReactClass: You're attempting to " +
-        'use a component as a mixin. Instead, just use a regular object.',
+        'use a component as a mixin. Instead, just use a regular object.'
     );
 
     var proto = Constructor.prototype;
@@ -492,7 +492,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
               'ReactClass: Unexpected spec policy %s for key %s ' +
                 'when mixing in component specs.',
               specPolicy,
-              name,
+              name
             );
 
             // For methods which are defined more than once, call the existing
@@ -534,7 +534,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
           'property, `%s`, that shouldn\'t be on the "statics" key. Define it ' +
           'as an instance property instead; it will still be accessible on the ' +
           'constructor.',
-        name,
+        name
       );
 
       var isInherited = name in Constructor;
@@ -543,7 +543,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
         'ReactClass: You are attempting to define ' +
           '`%s` on your component more than once. This conflict may be ' +
           'due to a mixin.',
-        name,
+        name
       );
       Constructor[name] = property;
     }
@@ -559,7 +559,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
   function mergeIntoWithNoDuplicateKeys(one, two) {
     _invariant(
       one && two && typeof one === 'object' && typeof two === 'object',
-      'mergeIntoWithNoDuplicateKeys(): Cannot merge non-objects.',
+      'mergeIntoWithNoDuplicateKeys(): Cannot merge non-objects.'
     );
 
     for (var key in two) {
@@ -571,7 +571,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
             'may be due to a mixin; in particular, this may be caused by two ' +
             'getInitialState() or getDefaultProps() methods returning objects ' +
             'with clashing keys.',
-          key,
+          key
         );
         one[key] = two[key];
       }
@@ -653,7 +653,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
               false,
               'bind(): React component methods may only be bound to the ' +
                 'component instance. See %s',
-              componentName,
+              componentName
             );
           }
         } else if (!args.length) {
@@ -663,7 +663,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
               'bind(): You are binding a component method to the component. ' +
                 'React does this for you automatically in a high-performance ' +
                 'way, so you can safely remove this call. See %s',
-              componentName,
+              componentName
             );
           }
           return boundMethod;
@@ -732,7 +732,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
             'prevent memory leaks.',
           (this.constructor && this.constructor.displayName) ||
             this.name ||
-            'Component',
+            'Component'
         );
         this.__didWarnIsMounted = true;
       }
@@ -744,7 +744,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
   _assign(
     ReactClassComponent.prototype,
     ReactComponent.prototype,
-    ReactClassMixin,
+    ReactClassMixin
   );
 
   /**
@@ -767,7 +767,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
         warning(
           this instanceof Constructor,
           'Something is calling a React component directly. Use a factory or ' +
-            'JSX instead. See: https://fb.me/react-legacyfactory',
+            'JSX instead. See: https://fb.me/react-legacyfactory'
         );
       }
 
@@ -801,7 +801,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
       _invariant(
         typeof initialState === 'object' && !Array.isArray(initialState),
         '%s.getInitialState(): must return an object or null',
-        Constructor.displayName || 'ReactCompositeComponent',
+        Constructor.displayName || 'ReactCompositeComponent'
       );
 
       this.state = initialState;
@@ -836,7 +836,7 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
 
     _invariant(
       Constructor.prototype.render,
-      'createClass(...): Class specification must implement a `render` method.',
+      'createClass(...): Class specification must implement a `render` method.'
     );
 
     if (process.env.NODE_ENV !== 'production') {
@@ -846,13 +846,13 @@ function factory(ReactComponent, isValidElement, ReactNoopUpdateQueue) {
           'componentShouldUpdate(). Did you mean shouldComponentUpdate()? ' +
           'The name is phrased as a question because the function is ' +
           'expected to return a value.',
-        spec.displayName || 'A component',
+        spec.displayName || 'A component'
       );
       warning(
         !Constructor.prototype.componentWillRecieveProps,
         '%s has a method called ' +
           'componentWillRecieveProps(). Did you mean componentWillReceiveProps()?',
-        spec.displayName || 'A component',
+        spec.displayName || 'A component'
       );
     }
 

addons/create-react-class/index.js

@@ -13,11 +13,18 @@
 var React = require('react');
 var factory = require('./factory');
 
+if (typeof React === 'undefined') {
+  throw Error(
+    'createReactClass could not find the React object. If you are using script tags, ' +
+      'make sure that React is being loaded before createReactClass.'
+  );
+}
+
 // Hack to grab NoopUpdateQueue from isomorphic React
 var ReactNoopUpdateQueue = new React.Component().updater;
 
 module.exports = factory(
   React.Component,
   React.isValidElement,
-  ReactNoopUpdateQueue,
+  ReactNoopUpdateQueue
 );

addons/create-react-class/package.json

@@ -26,13 +26,18 @@
     "object-assign": "^4.1.1"
   },
   "scripts": {
-    "test": "jest"
+    "test": "TEST_ENTRY=./index.js jest",
+    "build:dev": "NODE_ENV=development webpack && TEST_ENTRY=./create-react-class.js jest",
+    "build:prod": "NODE_ENV=production webpack && NODE_ENV=production TEST_ENTRY=./create-react-class.min.js jest",
+    "build": "npm run build:dev && npm run build:prod",
+    "prepublish": "npm test && npm run build"
   },
   "devDependencies": {
     "jest": "^19.0.2",
-    "react": "^15.5.0",
-    "react-addons-test-utils": "^15.5.0",
-    "react-dom": "^15.5.0"
+    "prop-types": "^15.5.10",
+    "react": "^15.5.4",
+    "react-dom": "^15.5.4",
+    "webpack": "^2.6.1"
   },
   "browserify": {
     "transform": [