The Wayback Machine - https://web.archive.org/web/20260225111349/https://github.com/docker/cli/pull/610
Skip to content

Comments

Update bash completion and deprecation for synchronous service updates#610

Merged
vdemeester merged 2 commits intodocker:masterfrom
albers:completion-service--detach
Oct 7, 2017
Merged

Update bash completion and deprecation for synchronous service updates#610
vdemeester merged 2 commits intodocker:masterfrom
albers:completion-service--detach

Conversation

@albers
Copy link
Collaborator

@albers albers commented Oct 7, 2017

The default value for docker service create|update|scale|rollback --detach|-d changed from true to false.
This updates bash completion to complete just --detach|-d.

This should go into 17.10.0 because the behavior changes in this release.

Also updates the deprecation of asynchronous service updates:

  • The default was not changed in 17.09 but will be in 17.10
  • service scale and service rollback are also affected.
albers added 2 commits October 7, 2017 13:57
The default value for
`docker service create|update|scale|rollback --detach|-d` changed from
`true` to `false`.
This updates bash completion to complete just `--detach|-d`.

Signed-off-by: Harald Albers <github@albersweb.de>
- The default was not changed in 17.09 but will be in 17.10
- `service scale` and `service rollback` are also affected.

Signed-off-by: Harald Albers <github@albersweb.de>
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, thank you!

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 🐯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment