pop
pop copied to clipboard
Upgrade jackc/pgx to v5
What is being done in this PR?
Closes https://github.com/gobuffalo/pop/issues/818
Supersedes https://github.com/gobuffalo/pop/pull/814
What are the main choices made to get to this solution?
jackc/pgx/v5 has slightly different handling of SQL NULLs than v4. Otherwise, this was a drop-in replacement.
List the manual test cases you've covered before sending this PR:
Ran tests against postgres locally.
pgx/v5 is a pretty big change from v4. I've changed this PR back to draft while waiting for some insights here: https://github.com/jackc/pgx/issues/1566
Superseded by https://github.com/gobuffalo/pop/pull/835