-
Notifications
You must be signed in to change notification settings - Fork 34
Update target frameworks #662
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
base: main
Are you sure you want to change the base?
Conversation
Wow, thanks! I had started to look at this here #638, and planned to go back and effectively make your PR (so I would go in stages update fwks changes, then take each feature as a follow PR). I think the gates will fail on your change, I just started a run - my other PR has fixes for that stuff in the build yaml files. I will take a proper look at this on the weekend. |
I updated the build scripts so that your changes can compile: #670. |
I merged the changes. |
Build is failing because deleting framework packages is not backwards compatible (i.e. a breaking change), so this would have to be a major release:
|
Is there going to be a major release soon? What's the plan? |
This pull request removes unsupported target frameworks (.NET Core, .NET 6.0) and simplifies conditional compilation so it supports
NETSTANDARD
(.NET Standard 2.0) andNET
(.NET 8.0 and .NET 9.0).