Use the namespaced function postgresql::postgresql_password()

[smortex]

Also include:

• Add support for Puppet 8 #185

[alexjfisher]

I don't think I'd go as far as marking this backwards-incompatible. IMO, it's probably fine to just bump the postgresql dependency in the metadata as you've done. This briefly came up on irc/slack the other day and IIRC @ekohl also took this position?? (If you look for them, I think you'll find plenty of other examples where we've not done a major release just because we required a newer version of a dependency)

[smortex]

I was thinking about it and as the next version in a major one, I decided that it was okay to mark it backwards-incompatible even if I truly believe nobody will be affected by this minimum version bump. I'll remove the label so that it still appears as "merged pull requests" in the changelog but without the emphasis it probably does not need.