Skip to content

Change to Docker Api#34

Open
ASDFGamer wants to merge 40 commits intomainfrom
docker_api
Open

Change to Docker Api#34
ASDFGamer wants to merge 40 commits intomainfrom
docker_api

Conversation

@ASDFGamer
Copy link
Copy Markdown
Member

This changes the interaction with docker from the cli to the api.
The API is better suited for this task, because it provides more and easier accessible information.
It also allows easier testing, because it can be mocked which wasn't the case for the cli calls.

This changes the behavior, so that all docker compose projects/services on the current host are found and not just ones that are in the root folder.
I think that I will add some way to limit the containers which are inspected by this program in a later PR

@ASDFGamer ASDFGamer self-assigned this Jul 6, 2024
@ASDFGamer ASDFGamer marked this pull request as draft July 6, 2024 21:24
@ASDFGamer ASDFGamer marked this pull request as ready for review July 7, 2024 21:07
@ASDFGamer ASDFGamer requested a review from ku-we July 7, 2024 21:12
@ASDFGamer
Copy link
Copy Markdown
Member Author

This closes #31

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant