Skip to content

Remove testutil - #918

Merged
thaJeztah merged 1 commit into
docker:masterfrom
dnephin:remove-testutil
Mar 6, 2018
Merged

thaJeztah merged 1 commit into
docker:masterfrom
dnephin:remove-testutil

Conversation

@dnephin

@dnephin dnephin commented Mar 6, 2018

Copy link
Copy Markdown
Contributor

replace with assert.ErrorContains

@codecov-io

codecov-io commented Mar 6, 2018 •

Copy link
Copy Markdown

Codecov Report

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

@@            Coverage Diff             @@
##           master     #918      +/-   ##
==========================================
- Coverage   53.55%   53.54%   -0.02%     
==========================================
  Files         262      262              
  Lines       16602    16602              
==========================================
- Hits         8891     8889       -2     
- Misses       7121     7123       +2     
  Partials      590      590

@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 🐯

Signed-off-by: Daniel Nephin <dnephin@docker.com>

@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 👍

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