-
Notifications
You must be signed in to change notification settings - Fork 18.6k
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 a note that remote and Boot2Docker users should not type sudo #9836
Add a note that remote and Boot2Docker users should not type sudo #9836
Conversation
Perhaps better to make a Nit; rename the include file to |
I'm always +1 for hyphens instead of underscores, especially since some In case you also wanted my opinion on organization, then I think I'm +1 on |
What @tianon said. |
@@ -0,0 +1,3 @@ | |||
> **Note:** if you are using a remote Docker daemon, such as Boot2Docker, | |||
> then _do not_ type the `sudo` before the `docker` commands shown in the | |||
> documentation examples. |
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.
documentation's examples.
Agreed re: hyphens preferable. Also agree with a |
@SvenDowideit ping pong |
Docker-DCO-1.1-Signed-off-by: Sven Dowideit <SvenDowideit@docker.com> (github: SvenDowideit)
e4b0265
to
d5df948
Compare
LGTM |
Thanks Sven! |
LGTM |
Add a note that remote and Boot2Docker users should not type sudo
Add a note that remote and Boot2Docker users should not type sudo (cherry picked from commit 71ab2ef) Docker-DCO-1.1-Signed-off-by: Michael Crosby <crosby.michael@gmail.com> (github: SvenDowideit)
Docker-DCO-1.1-Signed-off-by: Sven Dowideit SvenDowideit@docker.com (github: SvenDowideit)
Closes #9808
and I've implemented a quick transclusion so
{{ include "filename" }}
will be replaced by that content.