[openstreetmap/openstreetmap-website] Update Docker documentation and Dockerfile (PR #4101)
Andy Allan
notifications at github.com
Wed Jul 26 13:13:31 UTC 2023
@gravitystorm requested changes on this pull request.
A few things need addressing here:
* If compose v2 comes bundled with docker, then the installation instructions at the top of DOCKER.md need to be updated too.
* We want to keep the manual instructions and the CI docker tests in sync. So that means `.github/workflows/docker.yml` needs the same change from `docker-compose` to `docker compose`
* It might be worth removing the unrelated `ca-certificates` change from this PR, and resolve that separately.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4101#pullrequestreview-1547704830
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4101/review/1547704830 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20230726/bcb79295/attachment.htm>
More information about the rails-dev
mailing list