Skip to content

Fix bash completion for swarm init|join --data-path-addr#269

Merged
vdemeester merged 1 commit into
docker:masterfrom
albers:completion-swarm-init--data-path-addr
Jun 30, 2017
Merged

Fix bash completion for swarm init|join --data-path-addr#269
vdemeester merged 1 commit into
docker:masterfrom
albers:completion-swarm-init--data-path-addr

Conversation

@albers
Copy link
Copy Markdown
Collaborator

@albers albers commented Jun 29, 2017

bash completion for swarm init --data-path-addr and swarm join --data-path-addr treated --data-path-addr as a boolean option.
This fix adds completion for interfaces and local addresses, similar to --advertise-addr (but without trailing port).
Also corrects sort order of options.

Ping @sdurrheimer for zsh completion

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

codecov-io commented Jun 29, 2017

Codecov Report

Merging #269 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #269      +/-   ##
==========================================
+ Coverage   46.84%   46.85%   +<.01%     
==========================================
  Files         172      172              
  Lines       11692    11692              
==========================================
+ Hits         5477     5478       +1     
+ Misses       5903     5902       -1     
  Partials      312      312

Copy link
Copy Markdown
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

@thaJeztah
Copy link
Copy Markdown
Member

ping @vdemeester PTAL

Copy link
Copy Markdown
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 🐸

@vdemeester vdemeester merged commit 9689cca into docker:master Jun 30, 2017
@albers albers deleted the completion-swarm-init--data-path-addr branch June 30, 2017 08:30
nobiit pushed a commit to nobidev/docker-cli that referenced this pull request Nov 19, 2025
[17.10] update changelog for 17.10.0-ce-rc2
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