Skip to content

Conversation

@dreampiggy
Copy link
Collaborator

This fix #76

@codecov
Copy link

codecov bot commented Feb 1, 2020

Codecov Report

Merging #77 into master will increase coverage by 0.12%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #77      +/-   ##
==========================================
+ Coverage   72.49%   72.61%   +0.12%     
==========================================
  Files           9        9              
  Lines         669      672       +3     
==========================================
+ Hits          485      488       +3     
  Misses        184      184
Flag Coverage Δ
#ios 72.61% <100%> (+0.12%) ⬆️
Impacted Files Coverage Δ
SDWebImageSwiftUI/Classes/ImageManager.swift 74.46% <100%> (+0.55%) ⬆️
SDWebImageSwiftUI/Classes/WebImage.swift 82% <100%> (+0.24%) ⬆️

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 4336985...f56c39a. Read the comment docs.

@dreampiggy dreampiggy merged commit 7891347 into master Feb 1, 2020
@dreampiggy dreampiggy deleted the bugfix_onSuccess_when_memory_cache_hit branch February 1, 2020 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

onSuccess is not executed if already cached

2 participants