Fix multi option parsing

parcel-bundler · Nov 17, 2022 · 82136fb · 82136fb
1 parent a93ad5b
commit 82136fb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/packages/dev/parcel-link/bin/link.js b/packages/dev/parcel-link/bin/link.js
@@ -32,7 +32,6 @@ program
   .option(
     '-g, --node-modules-globs <globs...>',
     'Locations where node_modules should be linked in the app',
-    value => (Array.isArray(value) ? value : [value]),
     'node_modules',
   )
   .action((packageRoot, options) => {
@@ -41,7 +40,9 @@ program
       appRoot: process.cwd(),
       packageRoot: packageRoot ?? path.join(__dirname, '../../../'),
       namespace: options.namespace,
-      nodeModulesGlobs: options.nodeModulesGlobs,
+      nodeModulesGlobs: Array.isArray(options.nodeModulesGlobs)
+        ? options.nodeModulesGlobs
+        : [options.nodeModulesGlobs],
       dryRun: options.dryRun,
       log: console.log,
     });

diff --git a/packages/dev/parcel-link/bin/unlink.js b/packages/dev/parcel-link/bin/unlink.js
@@ -31,15 +31,16 @@ program
   .option(
     '-g, --node-modules-globs <globs...>',
     'Locations where node_modules should be unlinked in the app',
-    value => (Array.isArray(value) ? value : [value]),
     'node_modules',
   )
   .action((packageRoot, options) => {
     if (options.dryRun) console.log('Dry run...');
     unlink({
       appRoot: process.cwd(),
       namespace: options.namespace,
-      nodeModulesGlobs: options.nodeModulesGlobs,
+      nodeModulesGlobs: Array.isArray(options.nodeModulesGlobs)
+        ? options.nodeModulesGlobs
+        : [options.nodeModulesGlobs],
       dryRun: options.dryRun,
       forceInstall: options.forceInstall,
       log: console.log,