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

Update readme to work with newest Docker library #83

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tedivm
Copy link

@tedivm tedivm commented Oct 16, 2019

When docker ditched the docker-py library in favor of docker on PyPI they made a few very very small changes that break this library.

I've gone ahead and upgraded the readme to work with these new changes. I should point out that these changes are trivial- it basically amounts to changing docker.client() to docker.APIClient(). That's because the "low level API" that this project is based on has moved- but it is still there, and completely compatible with this library. Literally all anyone has to do in order to use the latest docker python library is change that one call so they get the low level client instead of the high level one.

resolves #78
resolves #79

When docker ditched the `docker-py` library in favor of `docker` on PyPI they made a few very very small changes that break this library.

I've gone ahead and upgraded the readme to work with these new changes. I should point out that these changes are trivial- it basically amounts to changing `docker.client()` to `docker.APIClient()`. That's because the "low level API" that this project is based on has moved- but it is still there, and completely compatible with this library. Literally all anyone has to do in order to use the latest docker python library is change that one call so they get the low level client instead of the high level one.
Copy link

@aureliojargas aureliojargas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirm that this worked for me using the latest dockerpty and docker SDK 4.1.0.

@acerv
Copy link

acerv commented Apr 13, 2020

Hello, Is there a plan to update the documentation?
At the moment, this is the only working library to get docker tty :)

Thanks for your work,
Andrea

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.

Dockerpty not compatible with docker version 3.0.0 dockerpty not compatible with docker 2.4.2
3 participants