Skip to content

Update docker build links #720

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update docker build links #720

wants to merge 1 commit into from

Conversation

willingc
Copy link
Contributor

@willingc willingc commented May 24, 2025

Currently the general container registry link redirects to files from 2022. This PR updates to the package location of builds for napari and napri-xpra.

Currently the general container registry link redirects to files from 2022. This PR updates to the package location of builds for napari and napri-xpra.
@github-actions github-actions bot added the documentation Improvements or additions to documentation label May 24, 2025
@psobolewskiPhD
Copy link
Member

Lol, i think the published date is when we started publishing packages. 🤯
If you click the links there, it goes to the same place.

Builds are available in the [GitHub Container Registry](https://github.com/orgs/napari/packages).
Builds are available in the GitHub Container Registry:
- [napari](https://github.com/napari/napari/pkgs/container/napari)
- [napari-xpra](https://github.com/napari/napari/pkgs/container/napari-xpra)

A dockerfile is added to napari root to allow build of a docker image using official napari release.
It contains two targets built on top of Ubuntu 20.04:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
It contains two targets built on top of Ubuntu 20.04:
It contains two targets built on top of the official Python 3.11 slim-bookworm image:

I'm not sure this is important. Maybe we just delete it so it's not something that needs to be maintained?

Similarly, below it should say:

  • napari: The result of `python -m pip install pip install "napari[all]", including all the ...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I can go ahead and review the rest of the doc. I just put the PR together when I saw the links 😉

Copy link
Member

Choose a reason for hiding this comment

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

Updating this is on my todo list 👀 (along with updating the xpra container), so I'd not worry about the whole document. But I just noticed those two lines as being out-dated. We could just remove them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants