dotnet / docs

This repository contains .NET Documentation.
https://learn.microsoft.com/dotnet
Creative Commons Attribution 4.0 International
4.13k stars 5.8k forks source link

Update Ubuntu install documentation (Ubuntu 24.04 LTS Noble Numbat - Cycle) #40612

Closed dviererbe closed 2 weeks ago

dviererbe commented 2 weeks ago

Summary


Internal previews

📄 File 🔗 Preview link
docs/core/install/linux-package-mixup.md Troubleshoot .NET errors related to missing files on Linux
docs/core/install/linux-ubuntu-install.md [.NET 8](#tab/dotnet8)
docs/core/install/linux-ubuntu.md Get OS version info
dviererbe commented 2 weeks ago

Force pushed to fix linting errors and do minor fixup.

MichaelSimons commented 2 weeks ago

FYI @richlander, @NikolaMilosavljevic

richlander commented 2 weeks ago

LGTM

Thanks! This is great to see.

There must be an include somewhere that is causing this misleading text with MS to show.

image

In a future revision, I'd like to remove all those dependency lists from the install docs and move them elsewhere. We should make every effort to simplify these docs. That is not today and thanks for updating them!

I noticed a glitch in "[R|r]untime" and "command[s]".

Look carefully at .NET 8

image

And then .NET 6

image

I realize that these are not related to the PR. I'm noticing them for the first time with the new version widget we have, which makes this type of inspection much easier.

richlander commented 2 weeks ago

@adegeo -- Is it possible to get this PR merged and live for tomorrow? We'd like it to be live for Ubuntu 24.04 launch, which (AFAIK) is 4/25.

adegeo commented 2 weeks ago

Sounds good. I'll follow up with fixes for any nits I find. Content looks great though! Thanks @dviererbe

richlander commented 2 weeks ago

Great. Can we push to live either today or in the morning?

adegeo commented 2 weeks ago

That happens automatically overnight or tomorrow morning, I can't remember which. We can force it at early afternoon if for some reason it didn't do it.

adegeo commented 2 weeks ago

Well it created the PR but didn't merge automatically, so I just merged it. Give it <30 and it will be live.

richlander commented 2 weeks ago

Thanks!