-
-
Notifications
You must be signed in to change notification settings - Fork 356
Add docs
requirements to pyproject.toml
#1494
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
Conversation
Toggling for RTD. Looks like the last build was unable to fetch the PR |
Weird RTD still can't fetch the PR |
Maybe this is upstream issue ( readthedocs/readthedocs.org#10601 ). Not sure why it started happening just now. Numcodecs didn't have the same issue. Also builds on |
Well at least RTD is reporting the status now, which it wasn't before. Let's see what upstream says |
Move the RTD exclusion to the `numpy` dependency directly.
@@ -13,7 +13,7 @@ maintainers = [ | |||
requires-python = ">=3.8" | |||
dependencies = [ | |||
'asciitree', | |||
'numpy>=1.20', | |||
'numpy>=1.20,!=1.21.0', |
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.
xref: numpy/numpy#19325
4e58e22
to
1d13385
Compare
'sphinx-rtd-theme', | ||
'pydata-sphinx-theme', | ||
'numpydoc', | ||
'numcodecs[msgpack]', |
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.
Since msgpack
is a numcodecs
requirement, specify it via numcodecs
' extra requirement
@joshmoore could you please review? This is some follow on work from issue ( #1492 ) |
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.
❤️ More of this goodness!
Instead of having a separate requirements file for RTD, fold these requirements into
pyproject.toml
as is done with Numcodecs.