SapientGuardian / mysql-connector-net-netstandard

ADO.NET driver for MySQL targeted against netstandard 1.3
GNU General Public License v2.0
46 stars 15 forks source link

Added MySql.Data.EntityFramework7 (EF7 Support) #26

Closed how02 closed 8 years ago

how02 commented 8 years ago

I converted original mysql's project MySql.Data.EntityFramework7 to dotnet core. Not entirely tested.

eByte23 commented 8 years ago

@how02 nice! only thing I see is that you'll need to change the project.json not to point to the direct reference of MySQL.Data as its build incremented. So possibly needs to change to "MySql.Data": "6.9.*" ?

SapientGuardian commented 8 years ago

Nice work! I'm happy to take this PR, but I wonder if it wouldn't better be suited to its own repo? It doesn't actually have any dependencies on files in this repo, beyond the base mysql library (which comes in via package anyway) does it?

IntelOrca commented 8 years ago

You've committed project.lock.json, this should be ignored from git.

eByte23 commented 8 years ago

True, but if oracle does want to pr this back might be nice to have it all in one? If it a standalone needs as I said will need version as 6.9.* and change the package name to SapientGuardian.MySql.Data

eByte23 commented 8 years ago

@how02 builds gonna fail because of not having target.

    "MySql.Data": {
      "version": "6.9.*",
      "target": "project"
    },

I believe

how02 commented 8 years ago

Woops sorry, I'm not very experienced with dotnet core.

eByte23 commented 8 years ago

All good, love the effort!

SapientGuardian commented 8 years ago

You're going to have to reference SapientGuardian.MySql.Data in your project.json rather than MySql.Data, because we rename the project at the last moment to change the package name. That leaves you with two effectively independent projects sitting in the same repo. Actually, it makes it even worse, because now I can't version the projects with the same number, since this one depends on the other having already been published.

In the unlikely event that Oracle wants to take what we've done (which would be awesome), they too should have split repos. Mono-repos are all kinds of terrible.

The more I think about this, the more I'm convinced that it should be separate.

eByte23 commented 8 years ago

Your right most people do breakout their projects into multiple repos, just thought maybe because of the oracle thing, but yes very unlikely. Also it seems like there is a weird bug in dotnet tooling as you "should" be able to reference as type:project the way we did when you have the folder in the global.json as a project but tested it doesn't seem to work either

SapientGuardian commented 8 years ago

@how02 how do you feel about splitting off your project into its own repo on your own account? We can help you set up ci and nuget if you need it. If you prefer, I can set it up here on my account as SapientGuardian.MySql.Data.EntityFramework7 and publish the nuget package as my own, but you did all the work and deserve all the credit :smile:.

eByte23 commented 8 years ago

Possibly consider a namespace change to line up with up naming convention to MySql.Data.EntityFrameworkCore

how02 commented 8 years ago

@SapientGuardian I think it would be easier if you set it up on your own account.

SapientGuardian commented 8 years ago

As you wish. I've moved your project to https://github.com/SapientGuardian/SapientGuardian.EntityFrameworkCore.MySql, squashing your commits in the process. I'd like to get a little testing done on it before I push it to NuGet, but the packages will be in the AppVeyor artifacts. Closing this PR; please do any subsequent work against the new repo. Thanks again for your efforts!