8000 Update docker build links by willingc · Pull Request #720 · napari/docs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
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

8000
Merged
merged 1 commit into from
Jun 23, 2025
Merged

Update docker build links #720

merged 1 commit into from
Jun 23, 2025

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.

@TimMonko TimMonko merged commit 7cb55b0 into napari:main Jun 23, 2025
9 of 10 checks passed
@TimMonko TimMonko added this to the 0.6.2 milestone Jun 23, 2025
@willingc willingc deleted the patch-3 branch June 24, 2025 06:26
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0