Skip to content
This repository was archived by the owner on Nov 12, 2023. It is now read-only.

fix: Consistent docker-flow for UnownHash projects#8

Merged
TurtIeSocks merged 1 commit intoUnownHash:mainfrom
clburlison:consistent-docker-flow
Mar 30, 2023
Merged

fix: Consistent docker-flow for UnownHash projects#8
TurtIeSocks merged 1 commit intoUnownHash:mainfrom
clburlison:consistent-docker-flow

Conversation

@clburlison
Copy link
Contributor

@clburlison clburlison commented Mar 30, 2023

Admin was the only project of the three not using CMD argument in the docker build. This aligns with FlyGOn & Golbat.

There are still some remaining inconsistencies between the projects but those are larger changes which I don't want to touch without knowing the final plan.

Compiles and runs on my system.

Fixes: UnownHash/Flygon#4

Copy link
Collaborator

@TurtIeSocks TurtIeSocks left a comment

Choose a reason for hiding this comment

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

Thanks!

@TurtIeSocks TurtIeSocks merged commit e79ba8b into UnownHash:main Mar 30, 2023
@clburlison clburlison deleted the consistent-docker-flow branch March 30, 2023 04:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docker-compose.yml command admin app missing

2 participants