Skip to content

Conversation

@hermanbanken
Copy link
Contributor

This pull request moves the examples so that they no longer have access to private methods in the rx.lang.scala package. These examples being in the same package already made TestScheduler.scala not work out of the box.

As a novice RxScala-test-writer 😅 I did not understand at first that some JavaConversion was going on, as I missed that the mock class was mocking the Java variant of Observer.

  • Move examples
  • Fix small missing java conversion in scaladoc

@samuelgruetter
Copy link
Collaborator

Cool, thanks :-)

But now all pages linking to https://github.com/ReactiveX/RxScala/blob/0.x/examples/src/test/scala/rx/lang/scala/examples/RxScalaDemo.scala will have to be updated. For instance the README or the RxScala website and probably many more...

I'd try to update as many as possible of them, but we'll probably miss some, so I'd suggest that you add a commit to this PR which adds an RxScalaDemo.scala file where it was before, containing only a comment with the new link.

* Move examples
* Fix small missing java conversion in scaladoc
@hermanbanken hermanbanken force-pushed the 0.x branch 3 times, most recently from baef021 to b89fd83 Compare May 31, 2015 09:21
RxScalaDemo was moved to not longer be in rx.lang.scala,
but references to this file may exist over the wiki and internet.
Therefore this file is kept to guide users where to find it now.

Fixes ReactiveX#167
@hermanbanken
Copy link
Contributor Author

@samuelgruetter I included a file at the original RxScalaDemo.scala's location and updated some Markdown files as well.

@samuelgruetter
Copy link
Collaborator

LGTM, thanks 👍

zsxwing added a commit that referenced this pull request Jun 1, 2015
@zsxwing zsxwing merged commit 4857c77 into ReactiveX:0.x Jun 1, 2015
@zsxwing
Copy link
Member

zsxwing commented Jun 1, 2015

Merged. Thank you.

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