Merge pull request #5 from streamich/fix-lerna

Fix lerna

lib/cli.js

@@ -21,28 +21,31 @@ const executeCommand = (command, args = [], env = process.env) => {
   });
 };
 
-// eslint-disable-next-line no-process-env
-const appendToExecuteCommand = (prepend) => (command, args = [], env = process.env) =>
-  executeCommand(command, [...args, ...prepend], env);
-
 const main = async () => {
   try {
     const state = createState();
 
-    await runInteractiveQuestions(state);
-
-    const message = formatCommitMessage(state);
-
     const {_: args, ...params} = minimist(process.argv.slice(2));
 
     for (const arg of args) {
       params[arg] = true;
     }
 
+    const {'dry-run': isDryRun, ...passThroughParams} = params;
+
+    if (isDryRun) {
+      // eslint-disable-next-line no-console
+      console.log('Running in dry mode.');
+    }
+
+    await runInteractiveQuestions(state);
+
+    const message = formatCommitMessage(state);
+
     const appendedArgs = [];
 
     // eslint-disable-next-line guard-for-in
-    for (const key in params) {
+    for (const key in passThroughParams) {
       const value = params[key];
 
       if (key.length === 1) {
@@ -56,9 +59,18 @@ const main = async () => {
       }
     }
 
-    const exec = appendToExecuteCommand(appendedArgs);
+    const executeCommandArgs = ['commit', '--message', message, ...appendedArgs];
+
+    if (isDryRun) {
+      const command = shellescape(['git', ...executeCommandArgs]);
 
-    exec('git', ['commit', '--message', message]);
+      // eslint-disable-next-line no-console
+      console.log('Will execute command:');
+      // eslint-disable-next-line no-console
+      console.log(command);
+    } else {
+      executeCommand('git', executeCommandArgs);
+    }
   } catch (error) {
     signale.fatal(error);
   }

lib/questions/lerna.js

@@ -6,8 +6,10 @@ exports.createQuestion = (state) => {
   }
 
   const changedPackages = getChangedPackages(state);
+  const allPackages = getAllPackages(state);
+
   const question = {
-    choices: getAllPackages(state),
+    choices: allPackages,
     default: changedPackages,
     message: `The packages that this commit has affected (${changedPackages.length} detected)\n`,
     name: 'packages',

lib/util/lerna.js

@@ -21,7 +21,7 @@ const getAllPackages = (state) => {
   try {
     const dir = path.join(state.root, 'packages');
 
-    return fs.readdirSync(dir).map(isDir(dir));
+    return fs.readdirSync(dir).filter(isDir(dir));
   } catch (error) {
     return [];
   }