Skip to content

[Swift] add RxSwift support#3490

Merged
wing328 merged 2 commits intoswagger-api:masterfrom
wwwjfy:swift-rxswift
Aug 25, 2016
Merged

[Swift] add RxSwift support#3490
wing328 merged 2 commits intoswagger-api:masterfrom
wwwjfy:swift-rxswift

Conversation

@wwwjfy
Copy link
Copy Markdown
Contributor

@wwwjfy wwwjfy commented Jul 31, 2016

No description provided.

@wwwjfy
Copy link
Copy Markdown
Contributor Author

wwwjfy commented Jul 31, 2016

I don't know what AppVeyor means by "AppVeyor was unable to build non-mergeable pull request", as Github says there is no conflict to merge

@wing328 wing328 closed this Jul 31, 2016
@wing328 wing328 reopened this Jul 31, 2016
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jul 31, 2016

Linked to #3477

@wwwjfy
Copy link
Copy Markdown
Contributor Author

wwwjfy commented Jul 31, 2016

I didn't add supportRxSwift as a CLI option described in #3477, but a responseAs option, because RxSwift is very much like PromiseKit, with different way for callbacks.

@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jul 31, 2016

@wwwjfy perfectly fine to use responseAs.

I've restarted the CI jobs and the AppVeyor has passed.

We'll review your change and let you know if we've any question.

Thanks for the contribution.

@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Jul 31, 2016

cc @Edubits @jaz-ah

@jaz-ah
Copy link
Copy Markdown
Contributor

jaz-ah commented Aug 4, 2016

+1 looks good to me @wing328

@wing328 wing328 modified the milestones: v2.2.1, v2.3.0, v2.2.2 Aug 8, 2016
@wing328
Copy link
Copy Markdown
Contributor

wing328 commented Aug 25, 2016

@wwwjfy all the local Swift tests passed. Thanks for the contribution.

@wing328 wing328 merged commit beaf1fc into swagger-api:master Aug 25, 2016
@wwwjfy wwwjfy deleted the swift-rxswift branch August 26, 2016 10:11
@wing328 wing328 changed the title fix #3477, add RxSwift support for Swift [Swift] add RxSwift support Feb 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants