-
Notifications
You must be signed in to change notification settings - Fork 696
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
Bump runnables to net9.0 #6240
base: dev
Are you sure you want to change the base?
Bump runnables to net9.0 #6240
Conversation
test/TestUtilities/Microsoft.Internal.NuGet.Testing.SignedPackages/Asn1/OcspSignature.cs
Show resolved
Hide resolved
You'll need to update this logic in the YAML: NuGet.Client/eng/pipelines/pr.yml Lines 178 to 195 in 2b3475a
I had been working on a way to make this more automatic but its hard and I didn't think we changed frameworks enough to warrant spending more time on it. There's probably more sections to change, let me know if you need help and I can push to your branch. |
Eventually, we'd want to go back to only a single Core framework anyways, so we can go back to ".NET tests' and ".NET Framework tests". |
Bug
Fixes:
Description
I followed https://github.com/NuGet/NuGet.Client/blob/dev/docs/updating-target-frameworks.md and here's the diff.
You can see the complete frameworks in the details below:
I also had to fix some obsolete constructor usage in the new utility project.
Followed the same pattern as #5911
PR Checklist
Added testsLink to an issue or pull request to update docs if this PR changes settings, environment variables, new feature, etc.Frameworks before:
Frameworks after: