-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Update dotnet to .NET 8.0 (LTS) #21877
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
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
igor-alexandrov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you!
offensive-vk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thankgod, finally someone did it !
dvdksn
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you 🫡
<!--Delete sections as needed --> ## Description The default port changed from 80 to 8080 when we switched to a newer version of .net in #21877. https://learn.microsoft.com/en-us/dotnet/core/compatibility/containers/8.0/aspnet-port https://deploy-preview-22269--docsdocker.netlify.app/guides/dotnet/develop/ https://deploy-preview-22269--docsdocker.netlify.app/guides/dotnet/deploy/ ## Related issues or tickets Closes #22265 Related to #22080 ENGDOCS-2492 ## Reviews <!-- Notes for reviewers here --> <!-- List applicable reviews (optionally @tag reviewers) --> - [ ] Technical review --------- Signed-off-by: Craig <[email protected]>
Description
Updated the dotnet tutorial to reference .NET 8 instead of .NET 6 (as .NET 6 is now unsupported and .NET is the current LTS version). I'm opening a parallel PR for the
docker-dotnet-samplerepo for the same update to keep these in sync.Reviews
Pretty straightforward changes. Mostly just changing 6's to 8's. No actual instructions needed changing. LOL