Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update README.md: change docker-compose to docker compose because command syntax has changed #1998

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

Geezus42
Copy link
Contributor

docker-compose --> docker compose

The official syntax of the command has changed. Many distributions and installers will automatically create an alias, but this should no be relied upon since it will not always be there. Documentation should be updated to reflect the standard syntax as defined by Docker.

$ docker-compose
The program 'docker-compose' is currently not installed. It is provided by
several packages. You can install it by typing one of the following:

$ docker compose
Usage:  docker compose [OPTIONS] COMMAND

`docker-compose` --> `docker compose`
@glensc
Copy link
Collaborator

glensc commented Jun 19, 2024

as written in #1995:

  • both should be supported.

@Geezus42
Copy link
Contributor Author

I don't understand the problem. You just referenced your own comment, which I already read, and did not provide any additional information. Docker has not said both will be supported. In fact, they ended support for Compose V1 in 2021.

The syntax of the command has officially been changed. If the old syntax works on your system, then it is a crutch that will eventually go away. It is better to simply learn the new syntax and update documentation to accordingly. Again, I do not understand the push back on such a minor change.

Can I still use Compose V1 if I want to?
image

@glensc glensc merged commit 09d8e5f into Taxel:main Jul 23, 2024
3 checks passed
@Geezus42 Geezus42 deleted the patch-2 branch July 23, 2024 12:52
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.

2 participants