Tweak wording and make package.json scripts less obnoxious

gaearon · gaearon · commit 2726f1d1c7c6 · 2016-07-17T12:58:13.000+01:00
diff --git a/bin/build-react-app.js b/bin/build-react-app.js
@@ -0,0 +1,2 @@
+#!/usr/bin/env node
+require('../scripts/build');
diff --git a/bin/graduate-react-app.js b/bin/graduate-react-app.js
@@ -0,0 +1,2 @@
+#!/usr/bin/env node
+require('../scripts/graduate');
diff --git a/bin/start-react-app.js b/bin/start-react-app.js
@@ -0,0 +1,2 @@
+#!/usr/bin/env node
+require('../scripts/start');
diff --git a/global-cli/index.js b/global-cli/index.js
@@ -77,9 +77,9 @@ function createApp(name, verbose, version) {
   var appName = path.basename(root);
 
   console.log(
-    'This will walk you through creating a new React app in',
-    root
+    'Creating a new React app in ' + root + '.'
   );
+  console.log();
 
   fs.mkdirSync(root);
 
@@ -92,6 +92,8 @@ function createApp(name, verbose, version) {
   process.chdir(root);
 
   console.log('Installing create-react-app-scripts package from npm...');
+  console.log('This might take a while! ⌛');
+  console.log();
 
   run(root, appName, version, verbose);
 }
diff --git a/package.json b/package.json
@@ -6,6 +6,11 @@
     "build": "node scripts/build.js local",
     "create-react-app": "node global-cli/index.js --scripts-version \"$PWD/`npm pack`\""
   },
+  "bin": {
+    "start-react-app": "./bin/start-react-app.js",
+    "build-react-app": "./bin/build-react-app.js",
+    "graduate-react-app": "./bin/graduate-react-app.js"
+  },
   "dependencies": {
     "autoprefixer": "^6.3.7",
     "babel-core": "^6.10.4",
diff --git a/scripts/init.js b/scripts/init.js
@@ -23,9 +23,9 @@ module.exports = function(hostPath, appName) {
 
   // Setup the script rules
   hostPackage.scripts = {};
-  ['start', 'build'].forEach(function(command) {
+  ['start', 'build', 'graduate'].forEach(function(command) {
     hostPackage.scripts[command] =
-      'node node_modules/create-react-app-scripts/scripts/' + command + '.js';
+      command + '-react-app';
   });
 
   fs.writeFileSync(hostPath + '/package.json', JSON.stringify(hostPackage, null, 2));
@@ -35,5 +35,16 @@ module.exports = function(hostPath, appName) {
   // Move the src folder
   fs.renameSync(selfPath + '/src', hostPath + '/src');
 
-  console.log('Creating the app', appName, 'at', hostPath);
+  console.log('Success! Created ' + appName + ' at ' + hostPath + '.');
+  console.log();
+  console.log('Inside that directory, you can run several commands:');
+  console.log('  * npm start: Starts the development server.');
+  console.log('  * npm run build: Builds the app for production.');
+  console.log('  * npm run graduate: Removes this tool. If you do this, you can’t go back!');
+  console.log();
+  console.log('We suggest that you begin by typing:');
+  console.log('  cd', appName);
+  console.log('  npm start');
+  console.log();
+  console.log('Happy hacking!');
 };

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+#!/usr/bin/env node`
	`2`	`+require('../scripts/build');`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+#!/usr/bin/env node`
	`2`	`+require('../scripts/graduate');`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+#!/usr/bin/env node`
	`2`	`+require('../scripts/start');`