Upgrade npgsql (Postgres) to the very latest version

Bartleby2718 commented 3 months ago

Context

Per the discussion in #111, this PR upgrades Npgsql from 5.0.4 to 8.0.2, the latest version at the time of filing.

Considerations

Breaking Changes

I don't think there are breaking changes that affect DistributedLock, but please take some time to go over the breaking changes:

in Npgsql 6: https://www.npgsql.org/doc/release-notes/6.0.html#breaking-changes
in Npgsql 7: https://www.npgsql.org/doc/release-notes/7.0.html#breaking-changes
in Npgsql 8: https://www.npgsql.org/doc/release-notes/8.0.html#breaking-changes

Usage of the Recent Versions

As you can see in https://www.nuget.org/packages/Npgsql/8.0.2#versions-body-tab, 8.0.0 has the most number of downloads, but both 8.0.1 and 8.0.2 have over a million downloads. Given that these are bug patches, I think 8.0.2 is better than the previous 8.0.* versions.

Open Questions

madelson commented 3 months ago

This looks good to go. By my count the only outstanding small comments are:

https://github.com/madelson/DistributedLock/pull/196/files#r1536824783 (if you wanted to try the approach suggested; can also be punted to a separate issue).
https://github.com/madelson/DistributedLock/pull/196/files#r1536825713 (minor cleanup)

Bartleby2718 commented 3 months ago

Leaving a comment so @madelson can assign me.

madelson / DistributedLock