feat:added a postgres version check

maplibre / martin

Blazing fast and lightweight PostGIS, MBtiles and PMtiles tile server, tile generation, and mbtiles tooling.

https://martin.maplibre.org

Apache License 2.0

2.33k stars 215 forks source link

feat:added a postgres version check #1427

Closed CommanderStorm closed 1 month ago

CommanderStorm commented 4 months ago

[x] adds a postgres version check
[x] added a testcase (using testcontainers) that the version is parsed correctly

Resolves #1230 (I think at this point the PR can be considered unasigned again, feel free to reject otherwise. Don't want to step on peoples toes ^^) Resolves #1229 (by using the minumum supported postgres version for postgis)

nyurik commented 1 month ago

thanks! test containers is an interesting feature, perhaps should use it in some other places too.

CommanderStorm commented 1 month ago

Bit of an overlooked downside: since it requires docker, the CI runners for Mac/windows would need to install this manually, but otherwise a really great tool for testing, can only recommend..