Skip to content

Add blocking docker create#6

Merged
gijzelaerr merged 2 commits into
gijzelaerr:masterfrom
kodsama:add_blocking_create
Feb 27, 2017
Merged

Add blocking docker create#6
gijzelaerr merged 2 commits into
gijzelaerr:masterfrom
kodsama:add_blocking_create

Conversation

@kodsama
Copy link
Copy Markdown
Contributor

@kodsama kodsama commented Feb 22, 2017

Added support for the docker create command.
It takes forever so that is why I added a _run_blocking definition, using the blocking approach is triggered by the blocking argument.

@gijzelaerr gijzelaerr self-assigned this Feb 23, 2017
@gijzelaerr
Copy link
Copy Markdown
Owner

awesome! looking good. Can you maybe extend the test suite so it tests the create command both blocking and not-blocking?

@kodsama
Copy link
Copy Markdown
Contributor Author

kodsama commented Feb 24, 2017

On it ;)

@kodsama
Copy link
Copy Markdown
Contributor Author

kodsama commented Feb 24, 2017

  Good enough ?
Can't test on the airport wifi but hope it passes ;)

@gijzelaerr
Copy link
Copy Markdown
Owner

perfect!

@gijzelaerr gijzelaerr merged commit b2a3806 into gijzelaerr:master Feb 27, 2017
@gijzelaerr
Copy link
Copy Markdown
Owner

made a new 0.2.2 release!

@kodsama
Copy link
Copy Markdown
Contributor Author

kodsama commented Feb 27, 2017

Great :)

@kodsama kodsama deleted the add_blocking_create branch February 27, 2017 11:38
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