Use docker compose instead of docker-compose in container-setup script#603
Use docker compose instead of docker-compose in container-setup script#603Blank-FS wants to merge 8 commits into
Conversation
This reverts commit 4c3d2a6.
|
Sorry, mb! I originally only intended for the PR to be the 1st commit. The 2nd commit was for personal use for everything to work locally on my end, but I forgot to switch a branch.
Please lmk if any of these other changes could potentially be useful, otherwise you can just consider the 1st commit for the PR. Thanks! |
`docker-compose` and `podman-compose` is obsolete since 2023, so those needed removing. The script didn't have correct error handling, so added it. Removed anti patterns too.
|
I've also fixed the outstanding issues of the script here. Now it should work better with newer versions of docker and podman. |
|
I've given it a go locally again, found one more issue and fixed it. Please try it on your end and if it works this is ready to merge. |
|
I tried it and found that Other than that, I was able get the script working. |
Changes container-setup script to use docker compose instead of docker-compose.
Newer developers will typically have docker compose installed.