datatogether / sentry

Parallelized web crawler written in Golang
GNU Affero General Public License v3.0
14 stars 6 forks source link

Docker usage #49

Closed machawk1 closed 6 years ago

machawk1 commented 6 years ago

Hey there, thank you for submitting an issue!

We are trying to keep issues for feature requests and bug reports. Please complete the following checklist before creating a new one:


The README specifies the command, "docker compose up" to install sentry via docker. My docker knowledge is a little rusty, but the aforementioned causes the error, "docker: 'compose' is not a docker command." to be returned. From what I recall, the binary is docker-compose, i.e., the command should be, docker-compose up with a very important hyphen.

I could be wrong or my local Docker could be updated, so please sanity check this before making the change to the README.

dcwalk commented 6 years ago

I think you are correct, docker-compose up is the command I use (and is in our other readmes, e.g., https://github.com/datatogether/webapp)

Thanks for the eagle eyes 🦅

dcwalk commented 6 years ago

@machawk1 , I made the small fix in #48 as that is also sprucing up README. I think once that is reviewed and merged we are good :)

machawk1 commented 6 years ago

@dcwalk Great! I also like the sound of the move from S3 to IPFS in #48. Looking forward to testing sentry further.

Closing this issue.