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

Xcode9 prototype issue fix #129

Merged
merged 3 commits into from
Dec 30, 2017
Merged

Xcode9 prototype issue fix #129

merged 3 commits into from
Dec 30, 2017

Conversation

andersio
Copy link
Member

@andersio andersio commented Dec 23, 2017

Resolves #115.
Preceded by #119 and #122.

@andersio andersio requested a review from mdiep December 23, 2017 10:41
@andersio andersio assigned andersio and unassigned andersio Dec 23, 2017
@andersio andersio force-pushed the xcode9-prototype-issue-fix branch from a33d25a to 837f82e Compare December 23, 2017 10:49
@andersio
Copy link
Member Author

@mdiep Since it could be source breaking, should we bump to 4.0? 🤔

@andersio andersio force-pushed the xcode9-prototype-issue-fix branch from 837f82e to d7263d0 Compare December 23, 2017 14:07
@mdiep
Copy link
Contributor

mdiep commented Dec 23, 2017

I don't think this is source breaking. The only change that could be is the EXTScope.h part; but that's an implementation detail. It shouldn't be considered public API.

@andersio
Copy link
Member Author

andersio commented Dec 24, 2017

I have changed the signature of zip:reduce: here though, which now has a generic returning closure like combineLatest.

@mdiep
Copy link
Contributor

mdiep commented Dec 25, 2017

Ah, I missed that bit. Still, I think that qualifies as a bug fix. Objective-C should be more tolerant of a change like that.

@andersio andersio merged commit eeaaefa into master Dec 30, 2017
@andersio andersio deleted the xcode9-prototype-issue-fix branch December 30, 2017 17:00
stuartofmine pushed a commit to stuartofmine/ReactiveObjC_2023 that referenced this pull request Oct 18, 2023
…ype-issue-fix

Xcode9 prototype issue fix
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