-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Doc update deps #23620
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 update deps #23620
Conversation
642f80d
to
a61807b
Compare
doc/devel/dependencies.rst
Outdated
.. _test-dependencies: | ||
|
||
Additional dependencies for testing | ||
=================================== | ||
----------------------------------- |
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.
This was intentionally not a subsection of the dev dependencies since you can run the tests without a dev setup. See note at https://matplotlib.org/stable/devel/testing.html#requirements.
While I’m not convinced that it’s necessary to support testing installed packages, we should keep the testing dependencies separate as long as we do that.
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.
hmm, fair enough. I'm not sure that I completely agree, but do not disagree anywhere near enough to argue about it.
d855f1b
to
431dc72
Compare
Also moves inkscape and gs to "optional" testing section because the tests will run with out them (but it will skip the ps / eps / svg tests) Co-authored-by: Tim Hoffmann <[email protected]>
and sort the list below Python
Co-authored-by: Oscar Gustafsson <[email protected]>
78739cf
to
9e7c049
Compare
ok, squashed down to reasonably sized commits. Can squash to one if people want. |
PR Summary
This is an update to our documented dependencies to match what the code does:
ack importorskip --ignore-dir=build --python
)