DOC update build dependencies#8374
Conversation
anntzer
left a comment
anntzer
left a comment
There was a problem hiding this comment.
feel free to dismiss once you consider the issues fixed.
doc/README.txt
Outdated
| To build the documentation, you will need additional dependencies: | ||
|
|
||
| All of these dependencies can be installed through pip:: | ||
| * Sphinx-1.3 or later(Version 1.5.0 is not supported) |
There was a problem hiding this comment.
missing space, I would also drop "Version" (or at least make it "version")
doc/README.txt
Outdated
| * Sphinx-1.3 or later(Version 1.5.0 is not supported) | ||
| * numpydoc 0.4 or later | ||
| * IPython | ||
| * Mock |
doc/README.txt
Outdated
| * Mock | ||
| * colorspacious | ||
| * pillow | ||
| * graphviz. |
There was a problem hiding this comment.
no final dot here, or add commas after other items (e.g. first answer at http://english.stackexchange.com/questions/55048/use-of-capitals-and-full-stops-in-bullet-points)
doc/README.txt
Outdated
| * pillow | ||
| * graphviz. | ||
|
|
||
| All of these dependencies except graphviz can be installed through pip:: |
There was a problem hiding this comment.
I would emphasize (*) "except graphviz".
|
Sorry, I don't have time to fix this. I'm closing this PR: this documentation problem will be perfect for a new contributor to work on. |
|
@NelleV Do you mind if I push up to this with a few changes? |
|
@phobson No, go for it. I reopened the PR and feel free to push. |
Missing word.
|
I fixed an additional typo. I think it should be good. |
|
Thanks! |
DOC update build dependencies
|
backported to v2.0.0-doc as 976692f |
5 participants
This complements #8373 and updates the README file to reflect graphviz is also a dependency.