Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix unquoted filepaths #43

Merged
merged 1 commit into from
Nov 15, 2019
Merged

Conversation

Js-Brecht
Copy link
Contributor

Pretty much guaranteed that there will be issues anytime a file path is passed to a shell command unquoted. If the path contains spaces, then the command will fail. See gatsbyjs/gatsby#16212 (on an older version)

These might be infrequent, but they happen. Most of them have already been fixed since the upgrade to v1.0.0. Here's a couple more.

@wardpeet
Copy link

@zetlen @davewasmer Do you need any extra information to move this forward? It's currently blocking an issue on gatsbyjs project.

@zetlen
Copy link
Collaborator

zetlen commented Nov 15, 2019

@wardpeet Hey, thanks for your patience. I'll investigate, but this shouldn't be a big deal.

@zetlen zetlen merged commit 6298379 into davewasmer:master Nov 15, 2019
@Js-Brecht Js-Brecht deleted the cli-command-issues branch November 22, 2019 20:38
alias-mac pushed a commit to alias-mac/devcert that referenced this pull request Feb 8, 2024
Bumps [yargs](https://github.com/yargs/yargs) from 15.4.1 to 16.2.0.
- [Release notes](https://github.com/yargs/yargs/releases)
- [Changelog](https://github.com/yargs/yargs/blob/master/CHANGELOG.md)
- [Commits](yargs/yargs@v15.4.1...v16.2.0)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants