-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Doc updates #9291
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
Doc updates #9291
Changes from 1 commit
3b52f44
296befb
10f78c4
ca0114f
87960f8
3ae1336
5562bf0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -49,9 +49,9 @@ installation of the package. In order to fully remove an installed Matplotlib: | |
Linux Notes | ||
=========== | ||
|
||
We recommend that you use your distribution's package manager to install | ||
Matplotlib. This will guarantee that Matplotlib's dependencies will be | ||
installed as well. | ||
To install Matplotlib at the system-level we recommend that you use your | ||
distribution's package manager to install Matplotlib. This will guarantee | ||
that Matplotlib's dependencies will be installed as well. | ||
|
||
If, for some reason, you can not use the package manager, you may use the | ||
manylinux wheels available on PyPI:: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. OK, when I read this I thought that There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. https://github.com/pypa/manylinux They never use quotes, I do not think we want to deviate from their notation. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's a bit jargon-esque but I'd prefer leaving it as it is. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not a big deal, but how about "...you may use the wheels available from manylinux, available on PyPI" There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sounds good! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Upon further throught I just dropped the "manylinux", as they are the only kind of linux wheels on pypi anyways, and either the reader knows what it means in which case it's obvious the wheels should be manylinux-tagged, or the reader doesn't and that doesn't help him at all. |
||
|
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.
Suggest drop: "...to install Matplotlib"