fahad19 / fahad19/glob-run

Escape dynamic path.

Open
#7 1 comment 0 reactions 0 assignees View on GitHub
bug Hacktoberfest help wanted
Dominant language
JavaScript
Stars
12
Forks
4
PR merge metrics
No merged PRs in 30d

Description

Cmd.exe and even Git Bash don't escape the dynamic part of the new command line:

`$ glob-run js-beautify --config H:\.jsbeautifyrc -r "**/*.html"`

```
>> js-beautify --config H:.jsbeautifyrc -r blocks/block-request-quotation.html
beautified blocks\block-request-quotation.html - unchanged
>> js-beautify --config H:.jsbeautifyrc -r blocks/block-return-packaging.html
beautified blocks\block-return-packaging.html - unchanged
>> js-beautify --config H:.jsbeautifyrc -r blocks/block-search-results (class selection).html

C:\Users\name\Documents\code\company\buildkit> "C:\Users\name\AppData\Roaming\npm\\node.exe" "C:\Users\name\AppData\Roaming\npm\\node_modules\js-beautify\js\bin\js-beautify.js" --config H:.jsbeautifyrc -r blocks/block-search-results (class selection).html
.html was unexpected at this time.
exec error: Error: Command failed: js-beautify --config H:.jsbeautifyrc -r blocks/block-search-results (class selection).html
.html was unexpected at this time.
```

`js-beautify -r "blocks/block-search-results (class selection).html"` should work cross-platform.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.