-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
CI Use lock files for CircleCI doc builds #23429
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
The full doc build succeeded see here |
# are using them for our min-dependencies doc generation. See | ||
# https://www.anaconda.com/why-we-removed-the-free-channel-in-conda-4-7/ for | ||
# more details. | ||
if [[ "$CIRCLE_JOB" == "doc-min-dependencies" ]]; then |
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 is from a while ago and only made sense when we were using the defaults channel (now using conda-forge) so I removed this
@@ -15,7 +15,7 @@ | |||
import warnings | |||
import re | |||
from datetime import datetime | |||
from packaging.version import parse |
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.
packaging
was installed in the environment, this is not needed since we are vendoring packaging
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.
Thank you for the PR!
"numpydoc", | ||
"sphinx-prompt", | ||
], | ||
# scikit-image min version (0.14.3) not available through conda-forge |
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.
Can we bump scikit-image to 0.16.2, which has a version on conda-forge for py38?
From scikit-image's pypi release history, 0.14.3 was released on 6/11/19, and 0.16.2 was on 10/22/19. Our min supported numpy 1.17.3 was released on 10/17/19.
I updated the min scikit-image version to 0.16.2. Both full doc builds succeeded: doc and doc-min-dependencies I'll fix the errors in the next commit (wrong version in README.rst). |
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.
LGTM
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.
LGTM as well!
Part of #22425