Skip to content

Conversation

@Hutch79
Copy link
Contributor

@Hutch79 Hutch79 commented Aug 1, 2025

I removed the build part in the example compose file which is located in the README.
Since an official docker image is published on GHCR and Docker Hub, this is unnecessary and hinders the user to update easily.

I also removed the version from the compose example, since this was deprecated.

Furthermore, I changed the published ports of all docker example to port 5000 which is the internal port, to eliminate the difference between docker and manual installations.

@PianoNic PianoNic requested review from PianoNic and Copilot August 1, 2025 22:42

This comment was marked as outdated.

@Hutch79 Hutch79 requested a review from Copilot August 1, 2025 22:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR streamlines the Docker Compose installation instructions by removing the local build process and using the official published Docker image instead. The changes simplify deployment by eliminating the need for users to build the image locally and standardize the port configuration across all Docker examples.

  • Removed the local build process from Docker Compose examples and switched to using the official published image
  • Standardized port mapping to 5000:5000 across all Docker examples for consistency
  • Removed deprecated version field from compose files and updated documentation

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
README.md Updated Docker Compose instructions to use official image instead of local build, standardized port mapping to 5000:5000, and removed deprecated Dockerfile example
compose.yml Added development usage disclaimer comment

@PianoNic
Copy link
Owner

PianoNic commented Aug 1, 2025

Nice Job 👍

@PianoNic PianoNic merged commit 222cd88 into PianoNic:main Aug 1, 2025
@PianoNic PianoNic added the enhancement Enhancement request label Aug 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants