rdagumampan / yuniql

Free and open source schema versioning and database migration made natively with .NET/6. NEW THIS MAY 2022! v1.3.15 released!
https://yuniql.io
Apache License 2.0
417 stars 63 forks source link

Why Core is linked to SqlServer ??? #241

Closed evilz closed 2 years ago

evilz commented 2 years ago

image

rdagumampan commented 2 years ago

Hi @evilz, thanks for reaching out. Good catch, I had look back again as I thought I have cut this before. Historically, SqlServer is the default driver and the only driver supported. But it has evolve to support more platforms so this should not have been the case.

This shows, the default factory method is still there. Marking this as for improvement. I will see if I can make this part of next release I'm cooking v1.2. Also feel free to PR if you can.

image

rdagumampan commented 2 years ago

Fixed. Dependency has been removed from Core and will be released as part of v1.2 https://github.com/rdagumampan/yuniql/commit/85b0ca37a1135e9b33a90358e8f8e508da3f0224