Skip to content
This repository has been archived by the owner on Jan 25, 2024. It is now read-only.

doc(readme) more info for Docker noobs #340

Closed
wants to merge 1 commit into from

Conversation

guillaumew
Copy link

I am not that used to using Docker. I had a few points where I was lost. I suggest some minor changes in the readme for people like me.

@guillaumew guillaumew added the documentation Improvements or additions to documentation label Dec 22, 2023
@@ -25,6 +25,16 @@ The simplest way to start with ecoindex-cli is to install docker and then create
alias ecoindex-cli="docker run -it --rm -v /tmp/ecoindex-cli:/tmp/ecoindex-cli vvatelot/ecoindex-cli:latest ecoindex-cli"
```

Launch docker-desktop
Copy link
Member

Choose a reason for hiding this comment

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

🎯 suggestion(non-blocking): Maybe we can precise here with a tip or something else that this section is for Windows users ? 🤔 What do you think ?

Copy link
Author

Choose a reason for hiding this comment

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

I add to do it on lubuntu as well...

@vvatelot vvatelot closed this Jan 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants