Skip to content
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 'rm' attribute to image resource. #46

Merged
merged 1 commit into from
Jan 20, 2014
Merged

Add 'rm' attribute to image resource. #46

merged 1 commit into from
Jan 20, 2014

Conversation

realloc
Copy link
Contributor

@realloc realloc commented Jan 20, 2014

Adding attribute to remove intermediate containers after a successful build. This makes things cleaner.

@bflad
Copy link
Contributor

bflad commented Jan 20, 2014

Awesome. Going to merge, fix up the alphabetization, and switch its implementation to the internal cli_args method instead of hardcoding it into that docker_cmd line. Will release in 0.26.0, which I should be able to get out with enhancement from #45 as well.

bflad added a commit that referenced this pull request Jan 20, 2014
Add 'rm' attribute to image resource.
@bflad bflad merged commit 2b9fda6 into sous-chefs:master Jan 20, 2014
bflad added a commit that referenced this pull request Jan 20, 2014
@bflad
Copy link
Contributor

bflad commented Jan 20, 2014

Its a good thing I switched this to cli_args, I think it would've printed -rm= on the command line each run (regardless of true/false).

@bflad
Copy link
Contributor

bflad commented Jan 21, 2014

Released in 0.26.0

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