-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Minor: Add installation link to README.md #8389
Conversation
docs/source/user-guide/cli.md
Outdated
@@ -46,6 +46,32 @@ brew install datafusion | |||
datafusion-cli | |||
``` | |||
|
|||
### Install and run using PyPI (on MacOS) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there other installations steps for other operating systems?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would just say leave the OS out and/or provide a link on ways how to install pip
docs/source/user-guide/cli.md
Outdated
# Downloading datafusion-33.0.0-cp38-abi3-macosx_11_0_arm64.whl.metadata (9.6 kB) | ||
# Collecting pyarrow>=11.0.0 (from datafusion) | ||
# Downloading pyarrow-14.0.1-cp39-cp39-macosx_11_0_arm64.whl.metadata (3.0 kB) | ||
# Requirement already satisfied: numpy>=1.16.6 in /Users/huangweijun/Library/Python/3.9/lib/python/site-packages (from pyarrow>=11.0.0->datafusion) (1.23.4) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should remove user name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm @Weijun-H
Thanks again
* Add installation link to README.md
Which issue does this PR close?
Closes #7297
Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?