Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolved the "-Wpartial-availability" warning. #158

Merged
merged 1 commit into from
Oct 12, 2017

Conversation

WilliamDenniss
Copy link
Member

@WilliamDenniss WilliamDenniss commented Oct 11, 2017

Implemented dismiss functionality for SFAuthenticationSession on iOS 11.
Removed GCC_TREAT_WARNINGS_AS_ERRORS=YES from the Podspec, enabled -Werror for TravisCI instead to catch new warnings at the library-developer level.
AppAuth now requires a Base SDK of iOS 11 (still supports iOS 7 as a Deployment Target).

Fixes #146

@codecov-io
Copy link

codecov-io commented Oct 11, 2017

Codecov Report

Merging #158 into master will decrease coverage by 0.01%.
The diff coverage is 96.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
- Coverage   80.11%   80.09%   -0.02%     
==========================================
  Files          41       41              
  Lines        3444     3447       +3     
  Branches      144      146       +2     
==========================================
+ Hits         2759     2761       +2     
- Misses        685      686       +1
Impacted Files Coverage Δ
Source/OIDURLQueryComponent.m 91.66% <96.87%> (-0.72%) ⬇️

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 c055052...1ffac3c. Read the comment docs.

Copy link
Collaborator

@tikurahul tikurahul left a comment

Choose a reason for hiding this comment

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

LGTM.

Implemented dismiss functionality for SFAuthenticationSession on iOS 11.
Removed GCC_TREAT_WARNINGS_AS_ERRORS=YES from the Podspec, enabled `-Werror` for TravisCI instead to catch new warnings at the library-developer level.
AppAuth now requires a Base SDK of iOS 11 (still supports iOS 7 as a Deployment Target).
@alexzchut
Copy link

@WilliamDenniss - Can you please make new tag for this change, because currently is it throwing errors on SFSafariViewController when integrating the framework. Thanks

shankari added a commit to shankari/cordova-jwt-auth that referenced this pull request Jan 18, 2018
Issue openid/AppAuth-iOS#146
Fix openid/AppAuth-iOS#158
Release in https://github.com/openid/AppAuth-iOS/releases/tag/0.91.0

> Fixes several "availability" warnings and errors. Corrects deployment targets.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants