-
Notifications
You must be signed in to change notification settings - Fork 673
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
Fixes #3484 (V1) - Updates SDK version to dotnet8, adds netstandard2.0
and net8.0
TargetFrameworks
#3486
Conversation
I would suggest adding, not replacing, the target for net8.0, because .net 7 projects will grab the netstandard version if there's not a package targeting 7. |
And actually, unless it causes build errors, dropping 2.1 to 2.0 will enable framework projects to load the netstandard version as well. 2.1 was for Mono and added some goodies, but didn't get backported to .net framework. If that does cause build errors, then of course leave 2.1 there for Mono peeps. But we probably shouldn't be breaking compatibility with existing systems for any changes made to v1 til it's fully retired. And adding multiple TFMs, if not build-breaking, costs us nothing. If an actual runtime issue crops up, we get to just say "sorry - not supported on EoL SDKs. Use at your own risk." |
netstandard2.0
and net8.0
TargetFrameworks
Fixes
Todos
dotnet8
netstandard2.0
to TFMsnet8.0
to TFMsClipboard.cs
Pull Request checklist:
CTRL-K-D
to automatically reformat your files before committing.dotnet test
before commit///
style comments)