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

ReactiveCocoa 4.0 support #83

Merged
merged 4 commits into from
Feb 2, 2016
Merged

ReactiveCocoa 4.0 support #83

merged 4 commits into from
Feb 2, 2016

Conversation

mbuchetics
Copy link
Contributor

Fixes a minor issues with the final ReactiveCocoa 4.0 release, which uses Result.NoError instead of its own NoError type. I also updated the Podspec and changed the version to 0.9.0.

@NachoSoto
Copy link

👍

@neilpa
Copy link
Member

neilpa commented Feb 2, 2016

NSTextField.swift also needs an import for Result.NoError

@mbuchetics
Copy link
Contributor Author

Sorry, forgot the Mac target. Should be fixed now.

@neilpa
Copy link
Member

neilpa commented Feb 2, 2016

Thanks!

neilpa added a commit that referenced this pull request Feb 2, 2016
ReactiveCocoa 4.0 support
@neilpa neilpa merged commit ee29c95 into RACCommunity:master Feb 2, 2016
@@ -18,8 +18,8 @@ Pod::Spec.new do |s|
s.watchos.deployment_target = '2.0'
s.tvos.deployment_target = '9.0'

s.source = { :git => 'https://github.com/neilpa/Rex.git', :tag => '0.9.0-RC.2' }
s.dependency 'ReactiveCocoa', '4.0.0-RC.2'
s.source = { :git => 'https://github.com/neilpa/Rex.git', :tag => '0.9.0' }
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm guessing you see into the future when @neilpa makes the actual tag? But on a serious note, could you so I can push a new spec to trunk? 👯

Copy link
Member

Choose a reason for hiding this comment

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

Heh. I started a new release right after I merged this https://github.com/neilpa/Rex/releases/tag/0.9.0

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, it's been a long time since I updated a podspec and the process has changed quite a bit. I'll try and get it pushed today though.

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 realized the other dependencies have no updated their specs either. But thanks for being so super quick. 👏

Copy link
Member

Choose a reason for hiding this comment

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

Looks like you can push this new spec too since you're marked as the maintainer in cocoapods.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, but the spec fails linting due to Result not being updated. So in either case we have to wait for the spec of Result 1.0.2 to be pushed.

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