-
Notifications
You must be signed in to change notification settings - Fork 643
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
move to 8.0.301 sdk #9986
move to 8.0.301 sdk #9986
Conversation
[ReleasePrep][2024.05.28]RI of dev into main
This seems reasonable, we just need to make sure our various CI and release pipelines are happy with this change. Thanks! |
Our release builds have some problem with this change. I think we would need to explicitly install .NET 8 to a special place, or something like that. It will take us some time to work out these changes (some of which may be closed source). Is this change blocking you? |
if u move to using the dotnet-install scripts it will respect your globa.json and install the correct version
.net 6 is 3 years old and i did not have it installed. imo it make sense to target the current stable, since there are security fixes and new features that are unlocked by using it |
can we merge this? |
@joelverhagen can we merge this? |
Sorry, it's quite busy period. Just pushed changes to internal release pipeline. Let's see if it fixes the problem. |
I created PR for internal release pipeline, hopefully I get approval by tomorrow. We need to merge them same time, otherwise things would break. |
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 for contribution.
Tracking issue: https://github.com/NuGet/Engineering/issues/5505