Skip to content
This repository has been archived by the owner on Apr 4, 2020. It is now read-only.

Add Carthage support #13

Merged
merged 4 commits into from
Jan 16, 2018
Merged

Conversation

alphatroya
Copy link
Contributor

No description provided.

@alphatroya
Copy link
Contributor Author

alphatroya commented Oct 25, 2017

The failed check was caused by old Swift syntax in the Example project. I think it will be solved when #12 will be closed

@mogol
Copy link
Member

mogol commented Oct 25, 2017

@alphatroya thanks for adding Carthage
I will merge it after #12

@mogol
Copy link
Member

mogol commented Jan 14, 2018

I have updated project to Swif4 and replaced SwiftWebSocket with Starscream. Could you pls update pr? Is it possible to add CI step to check that Carthage integration is not broken?

@alphatroya
Copy link
Contributor Author

I've update project for using Starscream framework. For check the carthage integration, I think carthage build --no-skip-current command will be enough.

@mogol mogol merged commit 2ef01ac into centrifugal:develop Jan 16, 2018
@mogol
Copy link
Member

mogol commented Jan 16, 2018

👍 thanks!

@alphatroya alphatroya deleted the feature/carthage branch January 16, 2018 10:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants