Skip to content

progress: Show success message when finished counting down#260

Merged
vdemeester merged 1 commit into
docker:masterfrom
aaronlehmann:verify-message
Jun 28, 2017
Merged

progress: Show success message when finished counting down#260
vdemeester merged 1 commit into
docker:masterfrom
aaronlehmann:verify-message

Conversation

@aaronlehmann

Copy link
Copy Markdown

This avoids leaving a message on the terminal saying there's 1 second remaining. See #237 (comment).

Open to suggestions on the wording.

cc @thaJeztah

Signed-off-by: Aaron Lehmann <aaron.lehmann@docker.com>
@thaJeztah

Copy link
Copy Markdown
Member

Verified this is working as expected;

$ docker service create --detach=false nginx:alpine
u7rp6ecohz0f3nvygihje5brr
overall progress: 1 out of 1 tasks
1/1: running   [==================================================>]
verify: Service converged

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

ping @vdemeester PTAL

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #260 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #260      +/-   ##
==========================================
- Coverage   46.85%   46.83%   -0.02%     
==========================================
  Files         172      172              
  Lines       11692    11692              
==========================================
- Hits         5478     5476       -2     
- Misses       5902     5904       +2     
  Partials      312      312

@vdemeester vdemeester left a comment

Copy link
Copy Markdown
Collaborator

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 92a2a1d into docker:master Jun 28, 2017
@GordonTheTurtle GordonTheTurtle added this to the 17.07.0 milestone Jun 28, 2017
@aaronlehmann aaronlehmann deleted the verify-message branch June 28, 2017 21:33
nobiit pushed a commit to nobidev/docker-cli that referenced this pull request Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment