Skip to content

Quote paths to handle spaces#152

Merged
ncalteen merged 1 commit intomainfrom
ncalteen/quotes
Feb 13, 2025
Merged

Quote paths to handle spaces#152
ncalteen merged 1 commit intomainfrom
ncalteen/quotes

Conversation

@ncalteen
Copy link
Copy Markdown
Collaborator

When setting the bootstrap path value, it was not quoted which could result in errors as seen in #148 when the path includes space characters. This PR fixes the issue by quoting the path value.

@ncalteen ncalteen self-assigned this Feb 13, 2025
Copilot AI review requested due to automatic review settings February 13, 2025 20:53
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

@ncalteen ncalteen merged commit 8f7aead into main Feb 13, 2025
@github-actions
Copy link
Copy Markdown

🦙 MegaLinter status: ❌ ERROR

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 4 0 0.07s
✅ JAVASCRIPT prettier 11 0 1.06s
✅ JSON jsonlint 24 0 0.2s
✅ JSON npm-package-json-lint yes no 2.85s
✅ JSON prettier 24 0 1.58s
✅ MARKDOWN markdownlint 8 0 3.1s
✅ REPOSITORY checkov yes no 26.29s
✅ REPOSITORY gitleaks yes no 2.72s
✅ REPOSITORY git_diff yes no 0.02s
✅ REPOSITORY grype yes no 21.97s
✅ REPOSITORY secretlint yes no 1.09s
❌ REPOSITORY trivy yes 1 12.63s
✅ REPOSITORY trivy-sbom yes no 1.19s
✅ REPOSITORY trufflehog yes no 28.14s
✅ TYPESCRIPT prettier 68 0 3.71s
✅ YAML prettier 21 0 1.13s
✅ YAML yamllint 21 0 0.81s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@ncalteen ncalteen deleted the ncalteen/quotes branch February 13, 2025 20:55
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.

2 participants