FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

MNT: Add missing dependency to environment.yml by timhoffm · Pull Request #28890 · matplotlib/matplotlib · GitHub

Repository navigation

MNT: Add missing dependency to environment.yml - #28890

Merged
QuLogic merged 1 commit into
mainfrom
mnt-envrionment-yml
Oct 18, 2024
Merged

QuLogic merged 1 commit into
mainfrom
mnt-envrionment-yml

Conversation

Copy link
Copy Markdown
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 added the PR: dependencies Pull requests that update a dependency file label Oct 16, 2024

Copy link
Copy Markdown
Member

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

QuLogic commented Oct 16, 2024

Copy link
Copy Markdown
Member

Should be fixed by cc8f05f

Copy link
Copy Markdown
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 force-pushed the mnt-envrionment-yml branch from 7c0ae8b to d8ccfbc Compare October 17, 2024 00:54
QuLogic added this to the v3.10.0 milestone Oct 18, 2024
QuLogic merged commit 3f42f35 into main Oct 18, 2024
QuLogic deleted the mnt-envrionment-yml branch October 18, 2024 20:28
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL