Merge pull request react-bootstrap#1020 from react-bootstrap/remove-f…

…actories

Remove factories

docs/src/GettingStartedPage.js

@@ -87,18 +87,6 @@ define(['react-bootstrap'], function(ReactBootstrap) { var Alert = ReactBootstra
                     />
                   </div>
 
-                  <h3>Without JSX</h3>
-                  <p>If you do not use JSX and just call components as functions, you must explicitly <a href="https://facebook.github.io/react/blog/2014/10/14/introducing-react-elements.html#deprecated-auto-generated-factories">create a factory before calling it</a>. React-bootstrap provides factories for you in <code>lib/factories</code>:</p>
-                  <div className="highlight">
-                    <CodeExample
-                      mode="javascript"
-                      codeText={
-`var Alert = require('react-bootstrap/lib/factories').Alert;
-// or
-var Alert = require('react-bootstrap/lib/factories/Alert');`
-                      }
-                    />
-                  </div>
                 </div>
                 <div className="bs-docs-section">
                   <h2 id="browser-support" className="page-header">Browser support</h2>

package.json

@@ -9,9 +9,8 @@
   "homepage": "http://react-bootstrap.github.io/",
   "scripts": {
     "build": "babel-node tools/build-cli.js",
-    "lib-build": "babel-node tools/build-cli.js --lib-only",
-    "test-watch": "npm run lib-build && karma start",
-    "test-coverage": "npm run lib-build && COVERAGE=true karma start --single-run",
+    "test-watch": "karma start",
+    "test-coverage": "COVERAGE=true karma start --single-run",
     "test": "npm run lint && npm run build && npm run tests-set",
     "tests-set": "karma start --single-run && _mocha --compilers js:babel-core/register test/server/*Spec.js",
     "lint": "eslint ./",

tools/build-cli.js

@@ -3,7 +3,6 @@
 import 'colors';
 import build from './build';
 import docs from '../docs/build';
-import lib from './lib/build';
 import { setExecOptions } from './exec';
 
 import yargs from 'yargs';
@@ -14,11 +13,6 @@ const argv = yargs
     demand: false,
     default: false
   })
-  .option('lib-only', {
-    demand: false,
-    default: false,
-    describe: 'Used for factories testing'
-  })
   .option('verbose', {
     demand: false,
     default: false,
@@ -35,9 +29,7 @@ setExecOptions(argv);
 
 let buildProcess;
 
-if (argv.libOnly) {
-  buildProcess = lib(argv);
-} else if (argv.docsOnly) {
+if (argv.docsOnly) {
   buildProcess = docs(argv);
 } else {
   buildProcess = build(argv);

tools/lib/build.js

@@ -2,17 +2,13 @@ import 'colors';
 import { exec } from '../exec';
 import fsp from 'fs-promise';
 import { srcRoot, libRoot } from '../constants';
-import generateFactories from '../generateFactories';
 import { buildFolder } from '../buildBabel';
 
 export default function BuildCommonJs() {
   console.log('Building: '.cyan + 'npm module'.green);
 
   return exec(`rimraf ${libRoot}`)
     .then(() => fsp.mkdirs(libRoot))
-    .then(() => Promise.all([
-      generateFactories(libRoot),
-      buildFolder(srcRoot, libRoot)
-    ]))
+    .then(() => buildFolder(srcRoot, libRoot))
     .then(() => console.log('Built: '.cyan + 'npm module'.green));
 }

tools/release-scripts/test.js

@@ -4,7 +4,7 @@ import { exec } from '../exec';
 function test() {
   console.log('Running: '.cyan + 'tests'.green);
 
-  return exec('npm run lib-build && npm run tests-set')
+  return exec('npm run tests-set')
     .then(() => console.log('Completed: '.cyan + 'tests'.green));
 }