Skip to content
This repository has been archived by the owner on Nov 6, 2019. It is now read-only.

Close ReadCloser #114

Merged
merged 1 commit into from
Sep 12, 2018
Merged

Close ReadCloser #114

merged 1 commit into from
Sep 12, 2018

Conversation

louisbranch
Copy link
Contributor

No description provided.

@louisbranch louisbranch self-assigned this Sep 12, 2018
@louisbranch louisbranch requested a review from a team September 12, 2018 17:37
Copy link
Contributor

@domenicrosati domenicrosati left a comment

Choose a reason for hiding this comment

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

Yes we need that!

@codecov
Copy link

codecov bot commented Sep 12, 2018

Codecov Report

Merging #114 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
+ Coverage   29.73%   29.76%   +0.02%     
==========================================
  Files          25       25              
  Lines        2559     2560       +1     
==========================================
+ Hits          761      762       +1     
  Misses       1733     1733              
  Partials       65       65
Impacted Files Coverage Δ
internal/registry/hub/docker_hub.go 71.64% <100%> (+0.42%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4d180ac...b51993a. Read the comment docs.

@louisbranch louisbranch merged commit 9e883ac into master Sep 12, 2018
@louisbranch louisbranch deleted the mem-leak branch September 12, 2018 18:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants