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

Add missing methods to Scala Adaptor #396

Merged
merged 20 commits into from
Sep 21, 2013

Conversation

samuelgruetter
Copy link
Contributor

This pull request adds a method to the Scala Observable for each method in the Java Observable. Some were renamed, have a different signature, or were left out intentionally. A comparison can be found here.

The Scala Observable should now be complete, but there are still some TODOs.

There should be no breaking changes, but only additions.

There's also an automatic completeness test (rx.lang.scala.CompletenessTest). If methods are missing, it still succeeds, but it prints the missing methods to stdout.

@cloudbees-pull-request-builder

RxJava-pull-requests #294 ABORTED

@samuelgruetter
Copy link
Contributor Author

So Jenkins needs all the 10 minutes to download scala-compiler-2.10.3-RC2.jar :D

benjchristensen added a commit that referenced this pull request Sep 21, 2013
Add missing methods to Scala Adaptor
@benjchristensen benjchristensen merged commit 0384095 into ReactiveX:master Sep 21, 2013
rickbw pushed a commit to rickbw/RxJava that referenced this pull request Jan 9, 2014
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