Thanks to visit codestin.com
Credit goes to github.com

Skip to content

MNT: Add missing dependency to environment.yml #28890

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

Merged
merged 1 commit into from
Oct 18, 2024
Merged

Conversation

timhoffm
Copy link
Member

#28617 introduced parallel gallery builds. This needs joblib as a dependency for sphinx-gallery. requirements.txt solves this via sphinx-gallery[parallel]>=0.12.0. Unfortunately, conda does not support variants, therefore we have to add the dependency explicitly to environment.yml.

@scottshambaugh scottshambaugh added the PR: dependencies Pull requests that update a dependency file label Oct 16, 2024
@tacaswell
Copy link
Member

Looks like the appveyor build (which uses this file) failed to do the pip install part of it.

@QuLogic
Copy link
Member

QuLogic commented Oct 16, 2024

Should be fixed by cc8f05f

@tacaswell
Copy link
Member

The bot says "no, this technically does not NEED a rebase" 🤣

#28617 introduced parallel gallery builds. This needs joblib as a
dependency for sphinx-gallery. `requirements.txt` solves this via
`sphinx-gallery[parallel]>=0.12.0`. Unfortunately, conda does not
support variants, therefore we have to add the dependency
explicitly to `environment.yml`.
@tacaswell tacaswell force-pushed the mnt-envrionment-yml branch from 7c0ae8b to d8ccfbc Compare October 17, 2024 00:54
@QuLogic QuLogic added this to the v3.10.0 milestone Oct 18, 2024
@QuLogic QuLogic merged commit 3f42f35 into main Oct 18, 2024
59 checks passed
@QuLogic QuLogic deleted the mnt-envrionment-yml branch October 18, 2024 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants