Skip to content

Fix bash completion for service update --force#1521

Merged
thaJeztah merged 1 commit intodocker:masterfrom
albers:completion-fix-service--force
Nov 15, 2018
Merged

Fix bash completion for service update --force#1521
thaJeztah merged 1 commit intodocker:masterfrom
albers:completion-fix-service--force

Conversation

@albers
Copy link
Copy Markdown
Collaborator

@albers albers commented Nov 15, 2018

- What I did
There are two issues with bash completion for docker service create|update --force (introduced in moby/moby#27596)

  • --force is not available in service create
  • --force is a boolean option

This PR fixes these issues.

- How to verify it

  • Verify that docker service create --fo<tab> does not complete to --force
  • Verify that docker service update --force --<tab> completes options

- `--force` is not available in `service create`
- `--force` is a boolean option

Signed-off-by: Harald Albers <github@albersweb.de>
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 🐯

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, thank you!

@thaJeztah thaJeztah merged commit f7ea8e8 into docker:master Nov 15, 2018
@GordonTheTurtle GordonTheTurtle added this to the 19.03.0 milestone Nov 15, 2018
@albers albers deleted the completion-fix-service--force branch November 15, 2018 19:16
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.

4 participants