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

Add bash completion for import --platform #1546

Merged
merged 1 commit into from
Dec 7, 2018

Conversation

albers
Copy link
Collaborator

@albers albers commented Dec 1, 2018

Bash completion for #1371.
The completion will only be added if the deamon runs in experimental mode.

Signed-off-by: Harald Albers <github@albersweb.de>
@codecov-io
Copy link

Codecov Report

Merging #1546 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1546   +/-   ##
=======================================
  Coverage   55.22%   55.22%           
=======================================
  Files         289      289           
  Lines       19389    19389           
=======================================
  Hits        10708    10708           
  Misses       7984     7984           
  Partials      697      697

1 similar comment
@codecov-io
Copy link

Codecov Report

Merging #1546 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1546   +/-   ##
=======================================
  Coverage   55.22%   55.22%           
=======================================
  Files         289      289           
  Lines       19389    19389           
=======================================
  Hits        10708    10708           
  Misses       7984     7984           
  Partials      697      697

Copy link
Collaborator

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🐯

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@thaJeztah thaJeztah merged commit a0fe333 into docker:master Dec 7, 2018
@GordonTheTurtle GordonTheTurtle added this to the 19.03.0 milestone Dec 7, 2018
@albers albers deleted the completion-import--platform branch December 7, 2018 20:14
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.

5 participants