Skip to content
This repository was archived by the owner on Feb 18, 2025. It is now read-only.

Conversation

@tapi
Copy link
Contributor

@tapi tapi commented Jun 10, 2015

Extension is now audited for nullability, assumes non null for the file as a whole and annotates specific items that are nullable.
should improve usage in Swift

Extension is now audited for nullability, assumes non null for the file as a whole and annotates specific items that are nullable.
should improve usage in Swift
@radazzouz radazzouz self-assigned this Jun 10, 2015
@radazzouz
Copy link
Contributor

Thank @tapi! 👍

@tapi
Copy link
Contributor Author

tapi commented Jun 10, 2015

Added a nil assertion in one of the methods since the documentation specified that it must be nonnull.

@tapi
Copy link
Contributor Author

tapi commented Jun 15, 2015

You're still supporting Xcode 5? I can add macro polyfills for the nullability stuff but it should work on Xcode 6 with iOS7 as a deployment target.

@radazzouz
Copy link
Contributor

Thanks for responding so quickly, @tapi! You're Awesome!

I use Xcode 5 to test compatibility with iOS 7.1 and earlier. I also tested with Xcode 6 and it all seems to work fine now. Merging right now!

Cheers!

radazzouz pushed a commit that referenced this pull request Jun 15, 2015
Audits extension for nullability
@radazzouz radazzouz merged commit 6f64dcf into agilebits:master Jun 15, 2015
@radazzouz radazzouz mentioned this pull request Aug 20, 2015
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