Update according to review in issue #1159#1165
Update according to review in issue #1159#1165benjchristensen merged 3 commits intoReactiveX:masterfrom
Conversation
|
RxJava-pull-requests #1080 SUCCESS |
There was a problem hiding this comment.
nError => onError ;-)
|
@benjchristensen could you wait for this PR ready and merge it before you release |
|
RxJava-pull-requests #1082 SUCCESS |
There was a problem hiding this comment.
You could differentiate here: instead of startWith(T), use +:, and instead of startWith(more than one T), use Observable.items(...) ++ o
There was a problem hiding this comment.
Thanks. Already updated it.
|
RxJava-pull-requests #1083 SUCCESS |
There was a problem hiding this comment.
The idea of this table is to have one row for each item, so you should split this into two. Also, when editing this code, from time to time you should run printMarkdownCorrespondenceTable, and convert the markdown to html and check if the generated html looks nice.
And you're also welcome to make PRs against https://github.com/RxScala/RxScala.github.io so that your updates are actually read by visitors of the http://rxscala.github.io/comparison.html page ;-)
There was a problem hiding this comment.
Updated it. I'll send a PR to RxScala.github.io once this one is merged.
|
RxJava-pull-requests #1084 SUCCESS |
Update according to review in issue #1159
/cc @samuelgruetter