postgres(normalize types to eliminate noop migrations)

HugoPeters1024 commented 1 year ago

Fixes #1517. I've kept the list non exhaustive because I simply was not sure how to handle every case. Notably when the alias did not have a placeholder variable but the base name did (e.g. timestamptz). I also looked around in the Postgresql.Simple modules to see if something like this already existed but it did not look like it.

At the risk of sounding selfish, this fixes my immediate problem does not make the code worse.

Before submitting your PR, check that you've:

[x] Ran stylish-haskell on any changed files.
[x] Adhered to the code style (see the .editorconfig file for details)

After submitting your PR:

[x] Update the Changelog.md file with a link to your PR
[x] Bumped the version number if there isn't an (unreleased) on the Changelog
[x] Check that CI passes (or if it fails, for reasons unrelated to your change, like CI timeouts)

parsonsmatt commented 1 year ago

This looks good to me. If you want to add a test and finish the PR checklist I'd be happy to do a patch release.

parsonsmatt commented 1 year ago

Released as persistent-postgresql-2.13.6.1

yesodweb / persistent

postgres(normalize types to eliminate noop migrations) #1518