Skip to content

Fix docker.client import.#1

Open
moigagoo wants to merge 3 commits into
xsteadfastx:masterfrom
moigagoo:patch-1
Open

Fix docker.client import.#1
moigagoo wants to merge 3 commits into
xsteadfastx:masterfrom
moigagoo:patch-1

Conversation

@moigagoo

@moigagoo moigagoo commented Jul 5, 2017

Copy link
Copy Markdown

Apparently, docker package got updated and Client was replaced with client.

Konstantin Molchanov added 3 commits July 5, 2017 12:15
Apparently, docker package got updated and `Client` was replaced with `client`.
@xsteadfastx

Copy link
Copy Markdown
Owner

are you actually using this? i thought its not needed anymore.

@moigagoo

Copy link
Copy Markdown
Author

@xsteadfastx Well, I use Docker a lot, and having autocompletions is very useful ¯_(ツ)_/¯

@xsteadfastx

Copy link
Copy Markdown
Owner

with what version did Client became APIClient. because the tests are failing

@moigagoo

Copy link
Copy Markdown
Author

lieryan added a commit to lieryan/dotfiles that referenced this pull request Jan 18, 2023
It's no longer needed, according to its own author
xsteadfastx/xonsh-docker-tabcomplete#1

Testing also shows that docker completion works even when this isn't
installed.
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