Adminer docker compose dev#1323
Conversation
kennethrioja
left a comment
There was a problem hiding this comment.
Minor comments, else it works on my side! Thanks Valentin, this is very useful!
|
|
||
| <http://localhost:3000> | ||
|
|
||
|
|
|
|
||
| ### Access Adminer | ||
|
|
||
| An Adminer instance is accessible at the following URL: |
There was a problem hiding this comment.
I would add the link to Adminer https://www.adminer.org/
and add something like 'An Adminer instance to connect to and visualize your local database...'
| volumes: | ||
| - db-data:/var/lib/postgresql/data | ||
| adminer: | ||
| image: adminer |
There was a problem hiding this comment.
Add the container name and a specific release tag of adminer, e.g.:
container_name: ${PREFIX}-adminer
image: adminer:5.4.2| image: adminer:5.4.2 | ||
| restart: always | ||
| ports: | ||
| - 8080:8080 |
There was a problem hiding this comment.
| - 8080:8080 | |
| - 8080:8080 | |
| depends_on: | |
| - db |
There was a problem hiding this comment.
This is one last suggestion I have before merging the PR
There was a problem hiding this comment.
Pull request overview
This PR updates the Docker-based development setup to make local database management easier by adding an Adminer service and documenting how to access it.
Changes:
- Adds an Adminer container to the development
docker-compose.yml, exposed on port 8080. - Documents Adminer access details in
docs/docker.md.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| docs/docker.md | Adds a short section describing how to access Adminer in dev. |
| docker-compose.yml | Introduces an adminer service alongside existing dev containers. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| adminer: | ||
| container_name: ${PREFIX}-adminer | ||
| image: adminer:5.4.2 |
| ports: | ||
| - 8080:8080 |
| adminer: | ||
| container_name: ${PREFIX}-adminer | ||
| image: adminer:5.4.2 | ||
| restart: always |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Summary of changes
Motivation and context
Checklist