Skip to content

Add missing methods to Scala Adaptor#396

Merged
benjchristensen merged 20 commits into
ReactiveX:masterfrom
samuelgruetter:idiomaticscala
Sep 21, 2013
Merged

Add missing methods to Scala Adaptor#396
benjchristensen merged 20 commits into
ReactiveX:masterfrom
samuelgruetter:idiomaticscala

Conversation

@samuelgruetter

Copy link
Copy Markdown
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

Copy link
Copy Markdown

RxJava-pull-requests #294 ABORTED

@samuelgruetter

Copy link
Copy Markdown
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