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

Harmonize CentOS support with doc site contents #8355

Merged
merged 1 commit into from
Apr 5, 2023
Merged

Conversation

leecow
Copy link
Member

@leecow leecow commented Apr 5, 2023

Support statements on the doc site distinguish between CentOS Linux and CentOS Stream. This update includes similar detail in the repo support content.

Resolves #7826

@leecow leecow requested a review from rbhanda as a code owner April 5, 2023 17:29
@leecow leecow requested review from richlander and rbhanda and removed request for rbhanda April 5, 2023 17:29
@mairaw mairaw merged commit 840621b into main Apr 5, 2023
4 of 5 checks passed
@mairaw mairaw deleted the update-centos branch April 5, 2023 17:53
@@ -31,11 +31,12 @@ OS | Version | Architectures
OS | Version | Architectures | Lifecycle
--------------------------------------|-----------------------|-------------------|----------
[Alpine Linux][Alpine] | 3.13+ | x64, Arm64, Arm32 | [Alpine][Alpine-lifecycle]
[CentOS][CentOS] | 7+ | x64 | [CentOS][CentOS-lifecycle]
[CentOS Linux][CentOS] | 7 | x64 | [CentOS][CentOS-lifecycle]
[CentOS Stream Linux][CentOS] | 8 | x64 | [CentOS][CentOS-lifecycle]
Copy link
Member

Choose a reason for hiding this comment

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

Should it be "8+"? CentOS Stream 9 exists too.

Copy link
Contributor

Choose a reason for hiding this comment

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

Goot catch. @leecow?

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be CentOS Stream instead of CentOS Stream Linux (we also don't write Ubuntu as Ubuntu Linux in the list) and 8+ for the version as also caught by @omajid for .NET 6 and .NET 7.

https://learn.microsoft.com/en-us/dotnet/core/install/linux-rhel#supported-distributions has it correctly.

Copy link
Contributor

Choose a reason for hiding this comment

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

For .NET 7, RHEL 7 is also still shown as supported. Since CentOS Stream is showing 8+, RHEL should also be fixed :) https://learn.microsoft.com/en-us/dotnet/core/install/linux-rhel#supported-distributions is already corrected to show RHEL 7 only supporting .NET 6.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, addressing comments in #8386

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update support docs on CentOS 8
5 participants