Skip to content
New issue

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

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

Already on GitHub? # to your account

Swift 4.2 Migration #45

Merged
merged 9 commits into from
Oct 16, 2018
Merged

Swift 4.2 Migration #45

merged 9 commits into from
Oct 16, 2018

Conversation

asowers1
Copy link
Collaborator

@asowers1 asowers1 commented Oct 9, 2018

What this does:

  • Updated for Swift 4.2 source compatibility
  • Updated project file to reflect that
  • Project test passing
  • Travis test passing

@asowers1 asowers1 changed the title Building on Swift 4.2 Swift 4.2 Migration Oct 9, 2018
@radex
Copy link
Owner

radex commented Oct 14, 2018

LGTM @asowers1! Thanks for the PR. Would you mind adding a changelog entry?

PS. I don't really maintain this repo anymore — I can give you GitHub and CocoaPods access if you want to take over

This was referenced Oct 14, 2018
@asowers1
Copy link
Collaborator Author

Sounds good to me! I can take care of that.

@asowers1
Copy link
Collaborator Author

@radex, the changelog is set. I can move ahead with the new tag and pod repo update whenever you're ready.

@radex
Copy link
Owner

radex commented Oct 16, 2018

@asowers1 Added you as GitHub collaborator and given you access to CocoaPods. Go ahead with pushing the changes. Feel free to update the readme to give yourself credit as the maintainer ;)

@asowers1 asowers1 merged commit 87ef0ed into radex:master Oct 16, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants