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 socket rocket to 0.4 #185

Merged
merged 5 commits into from
Sep 21, 2015
Merged

Update socket rocket to 0.4 #185

merged 5 commits into from
Sep 21, 2015

Conversation

hamchapman
Copy link
Contributor

All the tests still pass and I tried it out in the sample app and everything looked to be working as expected.

Also fixed a few other things as I came to them, as noted in the commit messages.

Updated iOS deployment target from 5.0 to 6.0.

Ran `pod install` with cocoapods 0.38.2 leading to lots of shuffling
around of files and renmaing of some files.

Removed some duplicated build phases for libPusher target.
@brandons
Copy link

The fix for _openConnection _closeConnection has been merged in SocketRocket.
facebookincubator/SocketRocket#263 Can you update your PR?

@lukeredpath
Copy link
Contributor

Looks good to me so far, we should also include in this PR a bump of the main library up to 1.6.1.

@hamchapman
Copy link
Contributor Author

Bumped the version of SocketRocket to 0.4.1 and the version of the pod itself to 1.6.1. Think I got all the references but could do with you checking it if possible @lukeredpath

lukeredpath added a commit that referenced this pull request Sep 21, 2015
@lukeredpath lukeredpath merged commit ce20b12 into pusher:master Sep 21, 2015
@lukeredpath
Copy link
Contributor

v1.6.1 now available via cocoapods, I still need to create a binary release though.

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