Use Microsoft.Data.SqlClient 2.1.0

[ErikEJ]

fixes #51
fixes #29

Design decisions:

• Name of package ?
• Use of strong naming (unclear why this was removed from the original package) ?

[dotMorten]

Use of signing (unclear why this was removed from the original package) ?

@ErikEJ what do you mean?

[ErikEJ]

Signing: you had signing enabled at some point, but then removed it soon after... Unsure why.

[dotMorten]

Signing: you had signing enabled at some point, but then removed it soon after... Unsure why.

I digitally sign the nuget packages locally at a later stage before publishing. I never did this here.

[ErikEJ]

Sorry, I mean strong naming!!

[dotMorten]

Aaah good call. Didn't even notice that went away

[ErikEJ]

I think it may be worth keeping it to improve the user experience (as far as I can tell from discussions in various GitHub issues)

[ErikEJ]

Should I add it back to the original project?

[ErikEJ]

Left strong naming enabled in the new project, should I also add it back to the original, or remove it altogether?

[ErikEJ]

Let me know if more changes are needed.
😄

[ErikEJ]

@dotMorten Anything I can do to help get this in?

[olmobrutall]

@ErikEJ since it looks like this is not moving forward, I've published a version using Microsoft.Data.SqlClient instead of System.Data.SqlClient in https://www.nuget.org/packages/Unofficial.Microsoft.SqlServer.Types

It's not based in your code (no #if NETCOREAPP3_1, just a hard change) but most of the changes are similar (removing namespaces etc..)