[added] Dry run and verbose options to release process

Add `--dry-run` and `--verbose` options to release process to facilitate
some means to test the release process. Also fixes a few bugs found in
the existing process along the way.

Closes react-bootstrap#563

Author: mtscout6

docs/build.js

@@ -5,7 +5,7 @@ import routes from './src/Routes';
 import Root from './src/Root';
 import fsp from 'fs-promise';
 import { copy } from '../tools/fs-utils';
-import { exec } from 'child-process-promise';
+import { exec } from '../tools/exec';
 
 const repoRoot = path.resolve(__dirname, '../');
 const docsBuilt = path.join(repoRoot, 'docs-built');

tools/amd/build.js

@@ -2,7 +2,7 @@ import _ from 'lodash';
 import path from 'path';
 import fsp from 'fs-promise';
 import { copy } from '../fs-utils';
-import { exec } from 'child-process-promise';
+import { exec } from '../exec';
 
 const repoRoot = path.resolve(__dirname, '../../');
 const amd = path.join(repoRoot, 'amd');

tools/build.js

@@ -7,6 +7,7 @@ import lib from './lib/build';
 import docs from '../docs/build';
 import dist from './dist/build';
 import { copy } from './fs-utils';
+import { setExecOptions } from './exec';
 
 import yargs from 'yargs';
 
@@ -19,8 +20,15 @@ const argv = yargs
     demand: false,
     default: false
   })
+  .option('verbose', {
+    demand: false,
+    default: false,
+    describe: 'Increased debug output'
+  })
   .argv;
 
+setExecOptions(argv);
+
 export default function Build(noExitOnFailure) {
   if (argv.docsOnly) {
     return docs();

tools/release-scripts/constants.js tools/constants.jstools/release-scripts/constants.js renamed to tools/constants.js

@@ -1,6 +1,6 @@
 import path from 'path';
 
-const repoRoot = path.resolve(__dirname, '../../');
+const repoRoot = path.resolve(__dirname, '../');
 
 const bowerRepo = 'git@github.com:react-bootstrap/react-bootstrap-bower.git';
 const docsRepo = 'git@github.com:react-bootstrap/react-bootstrap.github.io.git';

tools/dist/build.js

@@ -1,5 +1,5 @@
 import path from 'path';
-import { exec } from 'child-process-promise';
+import { exec } from '../exec';
 
 const repoRoot = path.resolve(__dirname, '../../');
 const dist = path.join(repoRoot, 'dist');

tools/exec.js

@@ -0,0 +1,62 @@
+import 'colors';
+import _ from 'lodash';
+import { exec } from 'child-process-promise';
+
+let executionOptions = {
+  dryRun: false,
+  verbose: false
+};
+
+function logWithPrefix(prefix, message) {
+  let formattedMessage = message.trim().split('\n')
+    .reduce((acc, line) => `${acc}${ acc !== '' ? '\n' : '' }${prefix} ${line}`, '');
+
+  console.log(formattedMessage);
+}
+
+function execWrapper(command, options = {}) {
+  let proc = exec(command, options);
+  let title = options.title || command;
+  let log = message => logWithPrefix(`[${title}]`.grey, message);
+
+  if (executionOptions.verbose) {
+    let output = (data, type) => {
+      logWithPrefix(`[${title}] ${type}:`.grey, data.toString());
+    };
+    proc = proc.progress(({stdout, stderr}) => {
+      stdout.on('data', data => output(data, 'stdout'));
+      stderr.on('data', data => output(data, 'stderr'));
+    })
+    .then(result => {
+      log('Complete'.cyan);
+      return result;
+    })
+    .catch(err => {
+      log(`ERROR: ${err.toString()}`.red);
+      throw err;
+    });
+  }
+
+  return proc;
+}
+
+function safeExec(command, options = {}) {
+  let title = options.title || command;
+
+  if (executionOptions.dryRun) {
+    logWithPrefix(`[${title}]`.grey, 'DRY RUN'.magenta);
+    return Promise.resolve();
+  }
+
+  return execWrapper(command, options);
+}
+
+function setExecOptions(options) {
+  executionOptions = _.extend({}, executionOptions, options);
+}
+
+export default {
+  exec: execWrapper,
+  safeExec,
+  setExecOptions
+};

tools/lib/build.js

@@ -1,6 +1,6 @@
 import 'colors';
 import path from 'path';
-import { exec } from 'child-process-promise';
+import { exec } from '../exec';
 
 const repoRoot = path.resolve(__dirname, '../../');
 const lib = path.join(repoRoot, 'lib');

tools/release-docs-scripts/release-docs.js

@@ -1,15 +1,33 @@
 /* eslint no-process-exit: 0 */
 
 import 'colors';
-import { exec } from 'child-process-promise';
+import yargs from 'yargs';
+import { exec, safeExec, setExecOptions } from '../exec';
 
 import preConditions from '../release-scripts/pre-conditions';
 import versionBump from '../release-scripts/version-bump';
 import repoRelease from '../release-scripts/repo-release';
 import tag from '../release-scripts/tag';
 import { lint } from '../release-scripts/test';
 
-import { repoRoot, docsRoot, docsRepo, tmpDocsRepo } from '../release-scripts/constants';
+import { repoRoot, docsRoot, docsRepo, tmpDocsRepo } from '../constants';
+
+const yargsConf = yargs
+  .usage('Usage: $0 [-n|--dry-run] [--verbose]')
+  .option('dry-run', {
+    alias: 'n',
+    demand: false,
+    default: false,
+    describe: 'Execute command in dry run mode. Will not commit, tag, push, or publish anything. Userful for testing.'
+  })
+  .option('verbose', {
+    demand: false,
+    default: false,
+    describe: 'Increased debug output'
+  });
+
+const argv = yargsConf.argv;
+setExecOptions(argv);
 
 let version;
 
@@ -22,7 +40,7 @@ preConditions()
   .then(versionBump(repoRoot, versionBumpOptions))
   .then(v => { version = v; })
   .then(() => {
-    return exec('npm run docs-build')
+    return exec(`npm run docs-build${ argv.verbose ? ' -- --verbose' : '' }`)
       .catch(err => {
         console.log('Docs-build failed, reverting version bump'.red);
         return exec('git reset HEAD .')
@@ -33,15 +51,19 @@ preConditions()
           });
       });
   })
-  .then(() => exec(`git commit -m "Release v${version}"`))
+  .then(() => safeExec(`git commit -m "Release v${version}"`))
   .then(() => Promise.all([
     tag(version),
     repoRelease(docsRepo, docsRoot, tmpDocsRepo, version)
   ]))
   .then(() => console.log('Version '.cyan + `v${version}`.green + ' released!'.cyan))
   .catch(err => {
     if (!err.__handled) {
-      console.error(err.message.red);
+      if (argv.verbose) {
+        console.error(err.stack.red);
+      } else {
+        console.error(err.toString().red);
+      }
     }
 
     process.exit(1);

tools/release-scripts/changelog.js

@@ -1,9 +1,9 @@
 import 'colors';
 import path from 'path';
-import { exec } from 'child-process-promise';
+import { exec, safeExec } from '../exec';
 
 export default (repoRoot, version) => {
   return exec(`node_modules/.bin/changelog -t v${version}`)
-    .then(() => exec(`git add ${path.join(repoRoot, 'CHANGELOG.md')}`))
+    .then(() => safeExec(`git add ${path.join(repoRoot, 'CHANGELOG.md')}`))
     .then(() => console.log('Generated Changelog'.cyan));
 };

tools/release-scripts/pre-conditions.js

@@ -1,5 +1,5 @@
 import 'colors';
-import { exec } from 'child-process-promise';
+import { exec } from '../exec';
 
 function ensureClean() {
   return exec('git diff-index --name-only HEAD --')