Skip to content

Push docker build and improve docker compose setup - #277

Open
miaooss wants to merge 10 commits into
mangostwo:masterfrom
miaooss:master
Open

miaooss wants to merge 10 commits into
mangostwo:masterfrom
miaooss:master

Conversation

@miaooss

@miaooss miaooss commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

I updated the Docker build to be published on GitHub.
This lets us reuse the build directly from Docker Compose.
I also updated the Docker Compose to reflect the changes.
I added some documentation.
I tested the whole project using this post as well:
https://www.getmangos.eu/wiki/documentation/installation-guides/guideslinux/installing-mangos-on-ubuntu-server-2204-r40014/


This change is Reviewable

@AppVeyorBot

Copy link
Copy Markdown

@miaooss
miaooss marked this pull request as draft September 16, 2026 02:36
@miaooss
miaooss marked this pull request as ready for review September 16, 2026 03:19
@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

@miaooss

miaooss commented Sep 17, 2026

Copy link
Copy Markdown
Contributor Author

@MadMaxMangos @billy1arm Any chance I can have a review of this?

@miaooss

miaooss commented Sep 17, 2026

Copy link
Copy Markdown
Contributor Author

We might also need an automatic cleanup of older builds.
https://jefftriplett.com/2024/github-actions-cleaning-up-old-docker-container-images/
or
https://github.com/snok/container-retention-policy

I can create a GitHub Action to do the cleanup too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants