Cast to string before passing input globs to slash

RyanZim · RyanZim · commit 4b9b66d5338b · 2020-12-03T12:54:07.000-05:00
If a number is passed as a positional argument, yargs interprets it as a JS Number, and slash tries to call .replace() on it, which fails. This ensures slash only gets strings passed to it. Refs #360 (comment)
diff --git a/index.js b/index.js
@@ -63,7 +63,10 @@ Promise.resolve()
     }
 
     if (input && input.length) {
-      return globber(input.map(slash), { dot: argv.includeDotfiles })
+      return globber(
+        input.map((i) => slash(String(i))),
+        { dot: argv.includeDotfiles }
+      )
     }
 
     if (argv.replace || argv.dir) {

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,10 @@ Promise.resolve()`
`63`	`63`	`}`
`64`	`64`
`65`	`65`	`if (input && input.length) {`
`66`		`- return globber(input.map(slash), { dot: argv.includeDotfiles })`
	`66`	`+ return globber(`
	`67`	`+ input.map((i) => slash(String(i))),`
	`68`	`+ { dot: argv.includeDotfiles }`
	`69`	`+ )`
`67`	`70`	`}`
`68`	`71`
`69`	`72`	`if (argv.replace \|\| argv.dir) {`