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

[WIP]: Added ReactiveCocoa-fied API Calls #106

Closed
wants to merge 10 commits into from
Closed

[WIP]: Added ReactiveCocoa-fied API Calls #106

wants to merge 10 commits into from

Conversation

czechboy0
Copy link
Member

Now all public API calls can be asked for a SignalProducer instead of just calling it immediately and passing a completion block.

Example

When getting the names of your bots, in addition to the existing call

server.getBots { (bots, error) -> ()
    if let bots = bots {
        print("My Bots: \(bots.map { $0.name })")
    }
}

you can instead call

server.getBots()
.map { $0.name }
.start(Event.sink(
                error: { print("Error: \($0)") },
                next: { print("My Bots Names: \($0)") }
            ))

Waiting For

@cojoj
Copy link
Contributor

cojoj commented Sep 14, 2015

@czechboy0 example you've provided doesn't look like RAC can bring something better 😜 We need to find more hardcore example 😁

Conflicts:
	Cartfile.resolved
	Podfile
	Podfile.lock
	XcodeServerSDK.xcodeproj/project.pbxproj
Conflicts:
	Podfile.lock
	XcodeServerSDK.podspec
@buildasaur
Copy link
Collaborator

Result of Integration 1

Duration: 1 minute and 38 seconds
Result: Perfect build! All 73 tests passed. 👍
Test Coverage: 54%.

@buildasaur
Copy link
Collaborator

Result of Integration 1

Duration: 1 minute and 59 seconds
Result: 8 errors, failing state: build-errors

@buildasaur
Copy link
Collaborator

Result of Integration 2

Duration: 2 minutes and 16 seconds
Result: 1 error, failing state: build-errors

@buildasaur
Copy link
Collaborator

Result of Integration 1

Duration: 2 minutes and 29 seconds
Result: 3 errors, failing state: build-errors

@czechboy0 czechboy0 closed this Mar 2, 2016
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.

3 participants