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

hotfix: RxSwift 4.0 and Carthage ketchup #55

Merged

Conversation

Gurpartap
Copy link
Contributor

catch up*

I've removed the missing HasDisposeBag.swift reference from the Xcode project so that the project compiles. Perhaps /pull/49 can suitably add it back.

@Gurpartap Gurpartap force-pushed the hotfix/carthage-rxswift-4 branch from c896539 to 79bb89e Compare October 26, 2017 11:19
@Gurpartap
Copy link
Contributor Author

$ cd ~/path/to/SomeProject/
$ cat Cartfile
github "Gurpartap/NSObject-Rx" "master"
$ carthage update --platform iOS --no-use-binaries
*** Fetching NSObject-Rx
*** Fetching RxSwift
*** Checking out RxSwift at "4.0.0"
*** Checking out NSObject-Rx at "79bb89e870ad7d10242069965c9d8f08e97c9378"
*** ...
*** Building scheme "RxSwift-iOS" in Rx.xcworkspace
*** Building scheme "NSObject_Rx" in NSObject-Rx.xcodeproj

69

Copy link
Member

@ashfurrow ashfurrow left a comment

Choose a reason for hiding this comment

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

Awesome, looks good. CI failure is unrelated, I'm fine with merging this – let me know what I can to do help.

@Gurpartap
Copy link
Contributor Author

The above output demonstrates the resolved RxSwift version after the PR patch, which is 4.0 instead of the incompatible 3.6.x before.

LGTM. Merge and release?

@ashfurrow ashfurrow merged commit d64b5ff into RxSwiftCommunity:master Oct 27, 2017
@ashfurrow
Copy link
Member

Thanks a lot for contributing! I've invited you to join the RxSwiftCommunity GitHub organization – no pressure to accept! If you'd like more information on what that means, check out our contributor guidelines and feel free to reach out with any questions.

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.

2 participants