8000 docs: add fix for Zsh globbing error with `pip install .[all]` by Chasing1020 · Pull Request #945 · ai-dynamo/dynamo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

docs: add fix for Zsh globbing error with pip install .[all] #945

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

Merged
merged 1 commit into from
May 7, 2025

Conversation

Chasing1020
Copy link
Contributor

Overview:

This PR updates the README.md to address the common error zsh: no matches found: .[all] when running pip install .[all] in Zsh.

Details:

Before this pr: zsh: no matches found: .[all].
Added explanations for why Zsh fails (square brackets interpreted as globbing). Recommended ".[all]" as the most portable fix.

Where should the reviewer start?

Verified commands work in both Zsh and Bash.

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • closes GitHub issue: #xxx

Signed-off-by: Chasing1020 <chasing1020@gmail.com>
Copy link
copy-pr-bot bot commented May 5, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Copy link
github-actions bot commented May 5, 2025

👋 Hi Chasing1020! Thank you for contributing to ai-dynamo/dynamo.

Just a reminder: The NVIDIA Test Github Validation CI runs an essential subset of the testing framework to quickly catch errors.Your PR reviewers may elect to test the changes comprehensively before approving your changes.

🚀

@github-actions github-actions bot added the external-contribution Pull request is from an external contributor label May 5, 2025
@ishandhanani ishandhanani merged commit 412ec84 into ai-dynamo:main May 7, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external-contribution Pull request is from an external contributor size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0