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 selecting tags documentation #6109

Merged
merged 10 commits into from
Dec 12, 2024
Merged

Conversation

lbussell
Copy link
Contributor

Attempting to fix Define doc providing guidance on Linux distros (#5063) while also cleaning up some stale documentation at the same time.

Related: Remove .NET 6 references from documentation (#6101)

> [!NOTE]
> Since .NET 8, these multi-platform tags **specifically exclude all Windows versions** due to `containerd`'s platform matching algorithm for Windows hosts.

Please see [#4492 (Switch multi-platform tags to Linux only)](https://github.com/dotnet/dotnet-docker/issues/4492) for more context.
Copy link
Member

Choose a reason for hiding this comment

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

The link to this issue is removed. I think it would be useful to keep this so that the community can easily find the context on why this decision was made for the .NET images.

samples/selecting-tags.md Outdated Show resolved Hide resolved
| Releases | [Every 6 months](https://alpinelinux.org/releases/) |
| Security | [Alpine Linux Security Tracker](https://security.alpinelinux.org/) |
| Support | [Alpine Linux Community](https://alpinelinux.org/community/) |
| Package manager | `apk` |
Copy link
Member

Choose a reason for hiding this comment

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

Would it be more useful to describe package type rather than package manager for each of these?


#### [Debian](https://www.debian.org/)

* Stability-focused, extensive package repository.
Copy link
Member

Choose a reason for hiding this comment

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

Describing this as stability-focused (and Ubuntu as "user-friendly") and having it limited to just that distro may be controversial.

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. There were already "opinions" in this doc and others. I would probably just prefer to remove all such opinions from our docs and only report facts. Users can make their own judgement.

@lbussell lbussell enabled auto-merge (squash) December 12, 2024 16:59
@lbussell lbussell merged commit 0a4258c into dotnet:main Dec 12, 2024
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Define doc providing guidance on Linux distros
2 participants