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

Updated linq2db.EntityFrameworkCore section with support EF Core 2 #3433

Merged
merged 1 commit into from
Oct 9, 2021

Conversation

sdanyliv
Copy link
Contributor

Just released version which supports EF Core 2.x

Just released version which supports EF Core 2.x
@@ -236,7 +236,7 @@ Adds hierarchyid support to the SQL Server EF Core provider. For EF Core: 3, 5.

### linq2db.EntityFrameworkCore

Alternative translator of LINQ queries to SQL expressions. For EF Core: 3, 5, 6.
Alternative translator of LINQ queries to SQL expressions. For EF Core: 2, 3, 5, 6.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But EF Core 2 is now out of support ??

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is out of support in ideal world. There are already big legacy projects which won't to be updated to 3.1.X or newer versions.

@smitpatel
Copy link
Contributor

Following #3392 we shouldn't merge this PR.

@ErikEJ
Copy link
Contributor

ErikEJ commented Sep 15, 2021

@smitpatel Why was this closed then?

#3354

@smitpatel
Copy link
Contributor

I think decision of #3392 is we want to remove where it causes confusion or force users to learn more things than they need to by leaving 2.x content but at the same time we want to preserve content for 2.x (where it is already written), where it could be useful and away from user. The extensions which have the version specified already can stay there (may be we will remove them at some point in future), there is certainly no need to add more content for 2.x at this stage.

@ErikEJ
Copy link
Contributor

ErikEJ commented Sep 15, 2021

Got it!

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