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

Change “Mac OS X” and “OS X” references to “macOS” #4104

Merged
merged 8 commits into from
Dec 15, 2017
Merged

Change “Mac OS X” and “OS X” references to “macOS” #4104

merged 8 commits into from
Dec 15, 2017

Conversation

SlaunchaMan
Copy link
Contributor

This PR:

  • Updates "OS X" and "Mac OS X" to "macOS"
  • Sets the warnings and such to Xcode 9.1 defaults.
  • Fixes some analyzer warnings about mutable copies and memory leaks.

@codecov-io
Copy link

codecov-io commented Nov 23, 2017

Codecov Report

Merging #4104 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4104      +/-   ##
==========================================
- Coverage    82.8%   82.79%   -0.01%     
==========================================
  Files          46       46              
  Lines        5304     5308       +4     
  Branches      439      441       +2     
==========================================
+ Hits         4392     4395       +3     
  Misses        671      671              
- Partials      241      242       +1
Impacted Files Coverage Δ
AFNetworking/AFURLRequestSerialization.m 60.03% <100%> (+0.12%) ⬆️
Tests/Tests/AFSecurityPolicyTests.m 100% <100%> (ø) ⬆️
Tests/Tests/AFHTTPSessionManagerTests.m 98.59% <100%> (ø) ⬆️
Tests/Tests/AFHTTPRequestSerializationTests.m 100% <100%> (ø) ⬆️
AFNetworking/AFURLSessionManager.m 59.89% <0%> (-0.19%) ⬇️

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 9d5a894...fc941b3. Read the comment docs.

@SlaunchaMan SlaunchaMan merged commit f7ecca5 into AFNetworking:master Dec 15, 2017
@SlaunchaMan SlaunchaMan deleted the 2017-updates branch December 15, 2017 15:38
@SlaunchaMan SlaunchaMan added this to the 3.2.0 milestone Dec 15, 2017
@SlaunchaMan SlaunchaMan changed the title 2017 Updates Change “Mac OS X” and “OS X” references to “macOS” Dec 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants