Npgsql 6 upgrade - Githubissues

JasperFx / weasel

Database Development Made Easy for .Net

MIT License

65 stars 21 forks source link

Npgsql 6 upgrade #35

Closed oskardudycz closed 2 years ago

oskardudycz commented 3 years ago

Added NpgsqlTypeMapping as a clone of the current Npgsql 6 mappings based on https://github.com/npgsql/npgsql/blob/a1d366c5692cc00a5edc1ec5bc9090952c7a63e7/src/Npgsql/TypeMapping/BuiltInTypeHandlerResolver.cs

See related PR in Marten: https://github.com/JasperFx/marten/pull/1987

mizrael commented 2 years ago

hi, I'm waiting for this PR to be merged, I would like to use Marten in a .NET6 project but it's currently not possible due to the dependency on Npgsql 5.0.10.

oskardudycz commented 2 years ago

@mizrael we want to add also a few other changes before this PR (it’ll take about a week or two). If you’d like to, I can create Marten prerelease version. P.S. Unless you explicitly reference Npgsql 6, you can already use Marten with .NET 6.

mizrael commented 2 years ago

thanks @oskardudycz . Unfortunately I am referencing Npgsql already in other parts of the project. I can totally wait for the final version (it's a pet project, no rush for now), in the meantime I could try to refactor and get rid of the dependency. thanks!

kieranbenton commented 2 years ago

If producing a pre release version is not too much trouble it would be very useful for us and would allow us to test in parallel.

samisq commented 2 years ago

I'd like to echo the same sentiment - if it's possible to publish a pre-release version of Marten with Npgsql v6 support, that would be much appreciated, as it would unblock our .net 6/npgsql 6 upgrade testing

oskardudycz commented 2 years ago

@samisq, @mizrael, @kieranbenton, I apologise for the long time not giving you alpha, but here it is: https://www.nuget.org/packages/Marten/5.0.0-alpha.3. Real v5 should be in the next few weeks.