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

Update for Swift 3 and xCode 8 #5

Merged
merged 5 commits into from
Oct 17, 2016
Merged

Update for Swift 3 and xCode 8 #5

merged 5 commits into from
Oct 17, 2016

Conversation

rudedogg
Copy link
Contributor

This update changes the API a little bit, which I mentioned in CHANGELOG.md. Because this isn't backwards compatible I bumped the version of the podspec to 1.0.0.

The pod lint passes without warnings, and the examples appear to work well. Let me know what you think, or if you'd like me to make any changes.

Thanks!

When contributing developers will need to run pod install before opening
the examples.
Updated source code and build settings to compile using Swift 3 and xCode 8.
API has been updated to more closely follow the Swift API Design
Guidelines. The setTheme() function has been set to have a discardable result to reduce
warnings.
@raspu
Copy link
Owner

raspu commented Oct 17, 2016

Hi! Thanks for all the effort you put here :). I was hopping on closing #1 before moving to 1.0, but I don't think I will be able to do it any time soon. So, I will merge your changes :).

@raspu raspu merged commit 41aa295 into raspu:master Oct 17, 2016
@raspu
Copy link
Owner

raspu commented Oct 17, 2016

I'll make a new release this week.

@raspu
Copy link
Owner

raspu commented Nov 2, 2016

@rudedogg Hey, I just sent you an invitation to become a collaborator. It would be nice if you can join as I have very little time to work on this project right now.

@rudedogg
Copy link
Contributor Author

rudedogg commented Nov 2, 2016

@raspu I'd love to. I'm doing some freelance work now, but I'd love to work on it in the near future, and can help with the admin stuff.

@raspu
Copy link
Owner

raspu commented Nov 3, 2016

@rudedogg Thanks! I really appreciate it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants