Skip to content

Update centos.md #392

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

Merged
merged 1 commit into from
Nov 1, 2016
Merged

Update centos.md #392

merged 1 commit into from
Nov 1, 2016

Conversation

xcellardoor
Copy link
Contributor

I've added in the '--rm' argument to the 'script method' and to the 'add your user to docker group' examples to match the first instruction. Purely a consistency thing, and so new users can appreciate not having a few hello-world containers spawned. Ta.

Applies to: Docker Documentation

I've added in the '--rm' argument to the 'script method' and to the 'add your user to docker group' examples to match the first instruction. Purely a consistency thing, and so new users can appreciate not having a few hello-world containers spawned. Ta.
@johndmulhausen
Copy link

Yep, I see that we start with using --rm then trail off. Thanks, @xcellardoor :)

@johndmulhausen johndmulhausen merged commit 28a16a7 into docker:master Nov 1, 2016
@johndmulhausen johndmulhausen self-assigned this Nov 1, 2016
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