Skip to content

add docker cp completion for container ids #2789

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

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

Conversation

bram2000
Copy link

- What I did
Modified the fish shell completion for docker cp such that it completes running container IDs as well as local file paths.

- How I did it
Added a new line to the competion file for fish, and edited a couple of the other cp completions to allow file paths.

- How to verify it

  1. Run a docker container
  2. Open fish shell
  3. Type docker cp followed by TAB to initiate completion
  4. Verify that file paths are completed, as well as the container ID of any running containers

- Description for the changelog
Update fish shell completion for docker cp to complete container IDs as well as file paths.

- A picture of a cute animal (not mandatory but encouraged)
Image of Hamster

Signed-off-by: Jon Bramley <brammers@gmail.com>
@bram2000 bram2000 force-pushed the complete-container-ids-for-docker-cp branch from 56f1a15 to 459825f Compare October 13, 2020 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants