Skip to content

Conversation

@thaJeztah
Copy link
Member

just some minor golint improvements 👍

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-io
Copy link

codecov-io commented Jan 13, 2018

Codecov Report

Merging #807 into master will increase coverage by 0.01%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #807      +/-   ##
==========================================
+ Coverage    50.9%   50.91%   +0.01%     
==========================================
  Files         237      237              
  Lines       15338    15332       -6     
==========================================
- Hits         7808     7807       -1     
+ Misses       7028     7025       -3     
+ Partials      502      500       -2

Copy link
Collaborator

@cpuguy83 cpuguy83 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
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 1694921 into docker:master Jan 13, 2018
@GordonTheTurtle GordonTheTurtle added this to the 18.02.0 milestone Jan 13, 2018
@thaJeztah thaJeztah deleted the golint-fixes branch January 13, 2018 14:04
nobiit pushed a commit to nobidev/docker-cli that referenced this pull request Nov 19, 2025
golint: remove redundant ifs
Upstream-commit: 1694921
Component: cli
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