Removed docker containers after execution, Fixes Issue #29

mserranom / bbrun

Run Bitbucket Pipelines locally

MIT License

208 stars 31 forks source link

Removed docker containers after execution, Fixes Issue #29 #31

Closed vijayakumark closed 6 years ago

vijayakumark commented 6 years ago

This patch adds --rm parameter to docker run to automatically remove containers after execution. Could help to avoid docker containers adding up and using disk space.

vijayakumark commented 6 years ago

@mserranom I see that the tests compare a snapshot of docker commands. Should the snapshots be modified too?

mserranom commented 6 years ago

Yeah, snapshots should be updated. I'm closing this PR and applying separately with the updated tests.

mserranom commented 6 years ago

https://github.com/mserranom/bbrun/pull/34 closing this