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
Updated links in INSTALL #4204
Updated links in INSTALL #4204
Conversation
Thanks. On a side note -- should we try to run the sphinx linkcheck on Travis to check this kind of stuff? @jenshnielsen ? |
@@ -243,7 +243,7 @@ ImageMagick | |||
Optional dependencies | |||
^^^^^^^^^^^^^^^^^^^^^ | |||
|
|||
`Pillow <http://python-imaging.github.io/>`__ | |||
`Pillow <https://github.com/python-pillow/Pillow>`__ | |||
If Pillow is installed, matplotlib can read and write a larger |
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.
Shouldn't this be http://python-pillow.github.io/ which looks like the homepage rather than the git repo?
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.
Absolutely...will update
It would make sense to me that Travis tests links in INSTALL etc. too. |
004552a
to
4092d3b
Compare
4092d3b
to
2f0ccaa
Compare
IMHO we should not hold this back on improving Travis to catch these issues. I have opened #4205 to remind my self to do that |
Updated dead links for pip and pillow in the INSTALL text