tazmainiandevil / dapper.abstractions Goto Github PK

A suggested enhancement would be to remove Microsoft.Data.SqlClient as a dependency or otherwise enable not having to pull in this library (and its own dependencies). This would allow projects that want to use the Abstractions but don't use SQL Server to not need to have the library pulled in.

.NET 7 is now out of support and .NET 6 LTS ends in November 2024

Great stuff - thanks for extending the functionality to include recent extension methods.

I started working with the library, only using Postgresql. I began writing my own PostgresExecutor, only to realize that if you change one thing, it should work with either:

In SqlExecutor, you require a SqlConnection to be passed in which only works with SQL Server. Why not pass in an IDbConnection? This lets me (a Postgresql+Dapper user) pass in whatever connection I have constructed, and the next guy (A MySql user) to pass in his connection.

Unless I am missing something big, IDbConnection has the same methods available from Dapper as SqlConnection (and as NpgsqlConnection, etc...).