-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Add docker and docker compose to dev install #4661
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -1,2 +1,4 @@ | |||
virtualenvwrapper | |||
pandoc | |||
docker-ce |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: leave a comment here indicating these are currently Rigetti only dependencies.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I add a comment here it will be necessary to strip these out when doing the apt-get xargs command that is described in development.md
I'll add a comment in development.md
We have docker tests and rigetti requires docker. This adds these to the dev apt-get list and explains that you need to install docker.
Missed comment in quantumlib#4661
We have docker tests and rigetti requires docker. This adds these to the dev apt-get list and explains that you need to install docker.