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

fix exhaustive switch error in Xcode 9 #30

Merged

Conversation

hirad
Copy link
Contributor

@hirad hirad commented Jul 14, 2017

Accounting for the new case in DispatchTimeInterval has been done in other ReactiveCocoa projects, but not this one.

@mdiep
Copy link
Contributor

mdiep commented Jul 15, 2017

This is great! Thanks for opening a PR!

@mdiep mdiep merged commit 332f39d into ReactiveCocoa:master Jul 15, 2017
@erwald
Copy link
Member

erwald commented Jul 17, 2017

@mdiep Any way we can make a 1.0.3 release with just this fix?
// Greetings from Xcode9b3 user. ;)

@mdiep
Copy link
Contributor

mdiep commented Jul 18, 2017

We can definitely do that.

Would you mind opening a PR that gets everything set for a 1.1 release?

  • Update ReactiveSwift in Cartfile to specify ~> 1.1.5
  • Update version numbers to 1.1
  • Update podspec versions

Once that's done and merged, then I can actually tag a release.

@erwald
Copy link
Member

erwald commented Jul 18, 2017

Sure! Branching it off of d527356 (the 1.0.2 commit)? Because we have other stuff on master which I suppose we don't want in.

@mdiep
Copy link
Contributor

mdiep commented Jul 18, 2017

I think the other things on master are okay. Everything looks additive to me.

@ikesyo
Copy link
Member

ikesyo commented Jul 18, 2017

Update ReactiveSwift in Cartfile to specify ~> 1.1.5

The requirement in podspec should also be updated.

@erwald
Copy link
Member

erwald commented Jul 18, 2017

All right, I'll do it tonight. 👍

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.

4 participants