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

fix(pull_test): --quiet option #2069

Merged
merged 1 commit into from
Aug 30, 2019
Merged

Conversation

knqyf263
Copy link
Contributor

Fix test of "docker pull --quiet"

Signed-off-by: Teppei Fukuda <knqyf263@gmail.com>
@codecov-io
Copy link

Codecov Report

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

@@            Coverage Diff             @@
##           master    #2069      +/-   ##
==========================================
+ Coverage   56.78%   56.79%   +<.01%     
==========================================
  Files         311      311              
  Lines       21836    21836              
==========================================
+ Hits        12400    12402       +2     
+ Misses       8520     8519       -1     
+ Partials      916      915       -1

1 similar comment
@codecov-io
Copy link

Codecov Report

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

@@            Coverage Diff             @@
##           master    #2069      +/-   ##
==========================================
+ Coverage   56.78%   56.79%   +<.01%     
==========================================
  Files         311      311              
  Lines       21836    21836              
==========================================
+ Hits        12400    12402       +2     
+ Misses       8520     8519       -1     
+ Partials      916      915       -1

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

nice find; looks like I spotted the issue in #882 (comment), but thought it was fixed in the test, but actually the .golden file was update to match the wrong output 😞 😅

Blame me for not spotting that in my final review 🤗

@thaJeztah
Copy link
Member

ping @vdemeester PTAL

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

@vdemeester vdemeester merged commit 5e99c05 into docker:master Aug 30, 2019
@GordonTheTurtle GordonTheTurtle added this to the 19.09.0 milestone Aug 30, 2019
@knqyf263 knqyf263 deleted the fix/pull_test branch August 30, 2019 13:10
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