Skip to content
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

Update build_and_deploy.yml to use napari docs dependency group #591

Merged
merged 2 commits into from
Feb 26, 2025

Conversation

psobolewskiPhD
Copy link
Member

References and relevant issues

Part of: #589
Depends on: napari/napari#7637

Description

Installs napari with [docs] and then doesn't run the docs-install step of the Makefile.

@github-actions github-actions bot added the maintenance CI, dependencies, and other maintenance label Feb 22, 2025
@psobolewskiPhD
Copy link
Member Author

This is failing because of the missing dependencies.
Not sure why CircleCI is hanging...

@psobolewskiPhD
Copy link
Member Author

napari/napari#7637 is merged, so re-running CI

Copy link
Contributor

@willingc willingc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @psobolewskiPhD. 🚢 so we can build via CI again.

jni pushed a commit to napari/napari that referenced this pull request Feb 24, 2025
# References and relevant issues
Part of: napari/docs#589
Depends on: napari/docs#591

# Description
This PR updates the build_docs.yml action to match napari/docs.
It's somewhat unneeded, because it ~fires off merge to main~ fires off
of `v*` tag push, while we build/deploy docs from napari/docs, but it
can be manually triggered, so it should be correct.
@jni jni merged commit 70a9965 into napari:main Feb 26, 2025
9 checks passed
DragaDoncila pushed a commit to napari/napari-sphinx-theme that referenced this pull request Feb 28, 2025
This PR updates the job that builds napari docs for the latest changes,
e.g. napari/docs#591
Additionally uses `html-noplot` to skip the gallery to make the run a
bit faster.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance CI, dependencies, and other maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants