mysticatea / npm-run-all

A CLI tool to run multiple npm-scripts in parallel or sequential.
MIT License
5.72k stars 240 forks source link

docs: recommend quoting for portability #260

Open dsherret opened 5 months ago

dsherret commented 5 months ago

The current recommendation relies on the user's shell's glob handling behaviour. It happens to work because generally something:* does not match any files, and so with sh's default behaviour it passes something:* as-is as an arg. This breaks down in some other shells though, for example, with zsh:

...or with yarn, as referenced in the issues that will be closed by this PR.

This change recommends quoting the globs with single quotes, which will avoid the glob expansion (which requires traversing the file system) and will also be more portable to work in more cases.

Closes https://github.com/mysticatea/npm-run-all/issues/200 Closes https://github.com/mysticatea/npm-run-all/issues/196