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

lien docker compose #1285

Merged
merged 3 commits into from
Nov 17, 2023
Merged

lien docker compose #1285

merged 3 commits into from
Nov 17, 2023

Conversation

bpeytraud
Copy link
Contributor

Ajout de lien pour réaliser l'installation de docker compose

@vaceletm vaceletm self-requested a review November 17, 2023 13:04
@vaceletm vaceletm self-assigned this Nov 17, 2023
@vaceletm
Copy link
Member

Change is OK but the commit message should be in english

@@ -16,7 +16,7 @@ It allows you to not be bothered by different OS migrations and the like.
Prerequisites
`````````````

You need docker on your host. You might want docker-compose as well.
You need docker on your host. You might want `docker-compose <https://docs.docker.com/engine/install/>`_ as well.
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't you link toward https://docs.docker.com/compose/ ?

Copy link
Member

Choose a reason for hiding this comment

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

Plus, I'm wondering why you added a link on docker compose but not on docker

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, because you want to have an explication of how to install docker compose depending of your environment. In my link you have all the command to do this.

@vaceletm vaceletm merged commit a0189af into Enalean:master Nov 17, 2023
3 checks passed
@bpeytraud bpeytraud deleted the guide-docker-intro branch January 5, 2024 13:42
@bpeytraud bpeytraud restored the guide-docker-intro branch January 5, 2024 13:42
@bpeytraud bpeytraud deleted the guide-docker-intro branch January 5, 2024 13:44
@bpeytraud bpeytraud restored the guide-docker-intro branch January 5, 2024 13:45
@bpeytraud bpeytraud deleted the guide-docker-intro branch January 5, 2024 13:46
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