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

Updated Quick, Nimble and ReactiveObjc for swift 4.2 support #55

Merged
merged 9 commits into from Apr 4, 2019
Merged

Updated Quick, Nimble and ReactiveObjc for swift 4.2 support #55

merged 9 commits into from Apr 4, 2019

Conversation

jakovi89
Copy link
Contributor

@jakovi89 jakovi89 commented Apr 1, 2019

  • Currently pointing at an intermediate branch for ReactiveObjc, supporting Swift 4.2
  • Updates Quick and Nimble to Swift 4.2 versions
  • Updates CI to Xcode 10.2

@ikesyo ikesyo mentioned this pull request Apr 1, 2019
@mdiep
Copy link
Contributor

mdiep commented Apr 1, 2019

CI failed. It looks like this PR still needs some work.

@jakovi89
Copy link
Contributor Author

jakovi89 commented Apr 2, 2019

hmm.. Watch OS still fails, I'm looking into it

@jakovi89
Copy link
Contributor Author

jakovi89 commented Apr 2, 2019

I'm somewhat in doubt why the Watch target will try to compile the UITextView+Rac on the CI, since these isn't part of the "compile sources". @mdiep do you have an idea why?

@jakovi89
Copy link
Contributor Author

jakovi89 commented Apr 2, 2019

It seems that the WatchOS target isn't choosing the correct target from ReactiveObjc

@mdiep
Copy link
Contributor

mdiep commented Apr 2, 2019

This is probably the old Xcode bug where it doesn't pick the correct platform for some reason. In the past, we've gotten around this by turning off implicit dependencies in the scheme and making the dependencies explicit.

@jakovi89
Copy link
Contributor Author

jakovi89 commented Apr 3, 2019

Thanks for the hint @mdiep 👍

Cartfile Outdated
@@ -1,2 +1,2 @@
github "ReactiveCocoa/ReactiveObjC" ~> 3.1.0
github "ReactiveCocoa/ReactiveSwift" ~> 4.0.0
github "ReactiveCocoa/ReactiveObjC" "27a7254837bb01122b51aac7c4c3961973351ca0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Actually, we should get an actual release here

Copy link
Contributor

Choose a reason for hiding this comment

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

I just released 3.1.1. Please update the Cartfile with that.
https://github.com/ReactiveCocoa/ReactiveObjC/releases/tag/3.1.1

@mdiep mdiep merged commit f376db7 into ReactiveCocoa:master Apr 4, 2019
@mdiep
Copy link
Contributor

mdiep commented Apr 4, 2019

Thanks! I made a new release with this: https://github.com/ReactiveCocoa/ReactiveObjCBridge/releases/tag/5.0.0

@jakovi89 jakovi89 deleted the swift-4.2 branch April 4, 2019 13:30
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.

None yet

2 participants