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

Fix docker-compose command #12994

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kanoshiou
Copy link
Contributor

This PR reverts #5681.

Since the README already instructs users to ensure that Docker Compose is installed, it is reasonable to assume that Docker Compose is already installed.

dify/README.md

Line 62 in d1ca492

The easiest way to start the Dify server is through [docker compose](docker/docker-compose.yaml). Before running Dify with the following commands, make sure that [Docker](https://docs.docker.com/get-docker/) and [Docker Compose](https://docs.docker.com/compose/install/) are installed on your machine:

Moreover, this is a backend document, and I believe it is essential to ensure the accuracy of the documentation. Especially since this is within a markdown code block, every line must run correctly (if there is an error, you should Google where the problem is or raise an issue).

And if they are indeed noobs, the original modification would actually lead them in the wrong direction.

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. 📚 documentation Improvements or additions to documentation labels Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📚 documentation Improvements or additions to documentation size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant