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

netstandard2.0 compatibility #271

Closed
Intecpsp opened this issue Oct 28, 2019 · 12 comments
Closed

netstandard2.0 compatibility #271

Intecpsp opened this issue Oct 28, 2019 · 12 comments
Milestone

Comments

@Intecpsp
Copy link

Does this project need to be on netstandard2.1? I'm using this package (v 8.0.0) without the option of upgrading to core 3.0 to get the new features and bug fixes. If there aren't any netstandard2.1/core3.0 features being used can the project be downgraded back to 2.0/2.2? Or have a separate branch that is built for those versions?

@SimonCropp
Copy link
Owner

EF Core 3.0 targets netstandard 2.1. so i needed to update to add compat for that version

@SimonCropp
Copy link
Owner

SimonCropp commented Oct 28, 2019

BTW if you are a user why are you not a patron? https://github.com/SimonCropp/GraphQL.EntityFramework#community-backed

@Intecpsp
Copy link
Author

But does the project need to be 3.0 compatible? It's brand new on the market and I don't see many enterprises making that migration for at least 6 months. Core 2.2 is the current mainstream standard, and since migrating to 3.0, anybody left on 2.2 won't get any new features or bug fixes for this library. As for becoming a patron, I'm actually using this library at the company I work at, so me being a patron doesn't make sense.

@Intecpsp
Copy link
Author

Intecpsp commented Oct 29, 2019

I actually just came across this: https://github.com/SLGShark6/GraphQL.EntityFramework/tree/core-2-compatible-branch

@SLGShark6 anyway to get this official?

@SimonCropp
Copy link
Owner

I'm actually using this library at the company I work at, so me being a patron doesn't make sense.

in that case all devs at your company should be patrons. https://github.com/SimonCropp/GraphQL.EntityFramework/blob/master/doco/licensing-patron-faq.md#organization-licensing

But does the project need to be 3.0 compatible?

yes it does. being v3 compat does not stop you using an older (v2 compat) version. but not being v3 compat stops any people wanting to use the current stable version in their apps. and there have been almost 400000 downloads of the v3 package, so there must be a non trivial number of real consumers.

I don't see many enterprises making that migration for at least 6 months

The enterprise org i work at has already upgraded and will be deploying this week

Or have a separate branch that is built for those versions?

This would be a reasonable amount of work. Unless more people become patrons, or someone offers funding to work on it, i cant justify the effort.

@Intecpsp
Copy link
Author

I will forward that licensing link to my superiors and have them figure out how to proceed from that standpoint. What about pulling in that core 2 branch I linked above?

@SimonCropp
Copy link
Owner

What about pulling in that core 2 branch I linked above?

Are you willing to put in the effort to keep a v2 branch updated? ie regularly pulling changes from master and submitting a PR to the v2 branch?

@Intecpsp
Copy link
Author

Intecpsp commented Oct 30, 2019 via email

@SimonCropp
Copy link
Owner

@Intecpsp that branch is 23 commits ahead, 41 commits behind master. and none of those changes are being submitted back as a releasable PR

how many developers are on your team?

@SimonCropp
Copy link
Owner

looks like ef 3.1 is going to target netstandard 2.0. so i will evaluate re-targeting netstandard 2.0
dotnet/efcore#18141

SimonCropp added a commit that referenced this issue Nov 17, 2019
@SimonCropp SimonCropp added this to the 9.3.0 milestone Nov 17, 2019
@SimonCropp
Copy link
Owner

i have released 9.3.0-beta.1 that targets EF3.1 and re adds netstandard 2 support

@SimonCropp
Copy link
Owner

there is now a stable on nuget that supports netstandard 2.0

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

No branches or pull requests

2 participants