docker-library / postgres

Docker Official Image packaging for Postgres
http://www.postgresql.org
MIT License
2.14k stars 1.11k forks source link

Link 'current' documentation page #1137

Open EugenKon opened 9 months ago

EugenKon commented 9 months ago

On the page:https://hub.docker.com/_/postgres in how to extend this image section:

Note 2: This variable defines the superuser password in the PostgreSQL instance, as set by the initdb script during initial container startup. It has no effect on the PGPASSWORD environment variable that may be used by the psql client at runtime, as described at https://www.postgresql.org/docs/14/libpq-envars.html. PGPASSWORD, if used, will be specified as a separate environment variable.

link to 14 postgres version is used. To my mind it would be better to use link to 'current' version https://www.postgresql.org/docs/current/libpq-envars.html

tianon commented 7 months ago

We avoid linking to "current" because as you can see, we're not the best at keeping up with updating URLs like this in our documentation, so when the upstream documentation reorganizes, our links break in ways that are harder to fix than if we have a link to something that was right and still works (and already links to newer versions of itself up at the top of the page).

tianon commented 7 months ago

To put that another way, I'd be in favor of a PR that updates https://github.com/docker-library/docs/blob/master/postgres/content.md?plain=1 to point to /16/, but not to one that updates to /current/.