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

Stop referencing Docker Hub repos in GitHub readmes and documentation #5905

Closed
MichaelSimons opened this issue Sep 23, 2024 · 2 comments · Fixed by #5936
Closed

Stop referencing Docker Hub repos in GitHub readmes and documentation #5905

MichaelSimons opened this issue Sep 23, 2024 · 2 comments · Fixed by #5936
Assignees

Comments

@MichaelSimons
Copy link
Member

In the past, the Docker Hub repo readmes were preferred because of the nicely formatted tags table. Due to size constraints these were removed. Because of that, it no longer makes sense to direct readers of the GitHub readmes and documentation to the Docker Hub readmes. Instead, the links should be updated to reference the appropriate GitHub readmes.

This is related to #5872 (comment)

@lbussell
Copy link
Contributor

[Triage] This was fixed for dotnet-docker in #5921

@MichaelSimons Please file a follow-up issue to fix this in the microsoft/dotnet-framework-docker repo as well.

@lbussell
Copy link
Contributor

[Triage] There are actually remaining references to DockerHub in the templated repo-specific readmes. For example:

* [dotnet](https://hub.docker.com/r/microsoft/dotnet/): .NET
* [dotnet/sdk](https://hub.docker.com/r/microsoft/dotnet-sdk/): .NET SDK
* [dotnet/runtime](https://hub.docker.com/r/microsoft/dotnet-runtime/): .NET Runtime
* [dotnet/runtime-deps](https://hub.docker.com/r/microsoft/dotnet-runtime-deps/): .NET Runtime Dependencies
* [dotnet/monitor](https://hub.docker.com/r/microsoft/dotnet-monitor/): .NET Monitor Tool
* [dotnet/aspire-dashboard](https://hub.docker.com/r/microsoft/dotnet-aspire-dashboard/): .NET Aspire Dashboard
* [dotnet/samples](https://hub.docker.com/r/microsoft/dotnet-samples/): .NET Samples
* [dotnet/nightly/aspnet](https://hub.docker.com/r/microsoft/dotnet-nightly-aspnet/): ASP.NET Core Runtime (Preview)

These should be fixed in a follow-up PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants