Skip to content
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

Target .NET 6.0 #25070

Merged
merged 1 commit into from
Jun 10, 2021
Merged

Target .NET 6.0 #25070

merged 1 commit into from
Jun 10, 2021

Conversation

bricelam
Copy link
Contributor

@bricelam bricelam commented Jun 9, 2021

Rationale:

  • Unlike .NET Standard, older versions of .NET don't enable any additional platforms.
  • We moved to .NET 5 because it enables the same platforms as .NET Standard 2.1, but with additional features (like savepoint APIs in ADO.NET) that we can leverage.
  • .NET 5 will go out of support three months after .NET 6 ships anyway.
  • The migration from .NET 5 to 6 should be pretty seamless

@bricelam bricelam requested a review from a team June 9, 2021 21:11
@bricelam bricelam merged commit 7105f84 into dotnet:main Jun 10, 2021
@bricelam bricelam deleted the net666 branch June 10, 2021 02:27
@ErikEJ
Copy link
Contributor

ErikEJ commented Jun 10, 2021

What is the milestone for this?

@roji
Copy link
Member

roji commented Jun 10, 2021

@ErikEJ this should apply for 6.0.0-preview6. Preview5 will still target net5.0.

@vslee
Copy link

vslee commented Jul 18, 2021

One downside to doing this in the preview phase is that it makes it a little more difficult for people to test the previews. Before, ppl could just update the NuGet from 5.0 to a 6.0.0-preview and try out the new EF Core. Now, it also requires installing a preview .NET Core.

@roji
Copy link
Member

roji commented Jul 18, 2021

@vslee that's true - we discussed this when making the decision to target net6.0. At the end of the day, we're starting to near the end of the 6.0 development cycle, and at some point we were bound to do this change. Hopefully installing a 6.0 dotnet SDK preview isn't that much of an effort.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants