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

Fix analyzer warning for upload task creation #4122

Merged
merged 3 commits into from Jan 26, 2018
Merged

Fix analyzer warning for upload task creation #4122

merged 3 commits into from Jan 26, 2018

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 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
3 participants