Skip to content
This repository was archived by the owner on Jan 17, 2023. It is now read-only.

Conversation

@SlaunchaMan
Copy link
Contributor

This is the same as @funnel20’s PR, but made off of the latest master so hopefully the coverage bot will have fewer changes to look at.

@codecov-io
Copy link

codecov-io commented Jan 6, 2018

Codecov Report

Merging #4122 into master will increase coverage by 0.18%.
The diff coverage is 84.21%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4122      +/-   ##
==========================================
+ Coverage   87.53%   87.72%   +0.18%     
==========================================
  Files          46       46              
  Lines        5690     5703      +13     
  Branches      292      295       +3     
==========================================
+ Hits         4981     5003      +22     
+ Misses        709      700       -9
Impacted Files Coverage Δ
Tests/Tests/AFURLSessionManagerTests.m 81.73% <100%> (+0.54%) ⬆️
AFNetworking/AFURLSessionManager.m 72.39% <66.66%> (+1.46%) ⬆️
AFNetworking/AFURLSessionManager.h 100% <0%> (+12.5%) ⬆️

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 f07fe50...67dd69e. Read the comment docs.

@SlaunchaMan SlaunchaMan requested a review from jshier January 12, 2018 18:32
@SlaunchaMan SlaunchaMan merged commit 78ce859 into master Jan 26, 2018
@SlaunchaMan SlaunchaMan deleted the merge-fix branch January 26, 2018 16:36
@SlaunchaMan SlaunchaMan added this to the 3.2.1 milestone May 4, 2018
@funnel20
Copy link

I can confirm this issue has been resolved in version 3.2.1. Thanks!

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.

4 participants