-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Docker images should point to mcr.microsoft instead of hub.docker #9454
Comments
I was planning to change them to this format instead: https://hub.docker.com/r/microsoft/dotnet-sdk/ @lbussell any comments on this? |
@mairaw the content on DockerHub is mostly static at this point. MAR at least has tag listings, e.g. https://mcr.microsoft.com/product/dotnet/sdk/tags. I am in favor of switching links to MAR. The link format would be https://mcr.microsoft.com/product/dotnet/sdk/. |
I don't like any of the options currently available. MAR has had some terrible delays in updating the tags table which provides a bad UX. The general UX of the tags table is poor. IMO the best tags browsing experience is the tags table in the GH readmes. Our GH repos still point to DH as the preferred portal because of the poor MAR UX. We should be consistent IMO. |
Agree
This was decided long before the reduced character limit on DockerHub forced us to remove the tags tables there. I think that MAR having any tags listing at all gives it a better user experience than DockerHub today (for .NET repos). Plus we (Microsoft) own MAR. I feel like the decision should be between MAR/GitHub. |
I vote for the links to point to the GitHub repo. |
I am onboard with GitHub links. Once the MAR portal is improved, we can switch to it. |
Ok, thanks all! I can help with the updates. |
PR created. I kept the https://hub.docker.com/_/microsoft-dotnet links. There were also links in the format https://hub.docker.com/_/microsoft-dotnet-core-runtime/ (with core in the url) that I applied the same logic. But did it in a separate commit in case my assumption was wrong. |
URL(s)
core/release-notes/9.0/preview/preview7/9.0.0-preview.7.md
Lines 61 to 67 in 7da6443
This also applies to previous .net versions
Description
The text was updated successfully, but these errors were encountered: