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

Add Swift 4 compatibility #108

Closed
wants to merge 1 commit into from

Conversation

jeffaburt
Copy link

@jeffaburt jeffaburt commented Sep 13, 2017

Title says it all!

Same end result as #98, but with no conflicts. Unfortunately I don't have write permissions to edit the code in that pull request.

After merging this PR, do you mind releasing a new version of Armchair on CocoaPods? Please and thanks!

@jeffaburt jeffaburt changed the title Add swift 4 compatibility Add Swift 4 compatibility Sep 13, 2017
Copy link
Member

@coneybeare coneybeare left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@jeffaburt
Copy link
Author

There were two additional places I had to include @objc. Please review this again when you're able to. Thanks!

@coneybeare
Copy link
Member

These changes are already in Master.

@coneybeare coneybeare closed this Sep 26, 2017
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.

3 participants