Skip to content

2.x: add javadoc and unit test to the recently changed XTransformers#4674

Merged
akarnokd merged 2 commits intoReactiveX:2.xfrom
akarnokd:TransformerCleanup
Oct 6, 2016
Merged

2.x: add javadoc and unit test to the recently changed XTransformers#4674
akarnokd merged 2 commits intoReactiveX:2.xfrom
akarnokd:TransformerCleanup

Conversation

@akarnokd
Copy link
Member

@akarnokd akarnokd commented Oct 6, 2016

This PR adds the javadoc and crash-testing for each XTransformer interface changed by #4672.

@akarnokd akarnokd added this to the 2.0 RC4 milestone Oct 6, 2016
@akarnokd akarnokd changed the title 2.x: add javadoc and unit test to the recently changed XTransformer 2.x: add javadoc and unit test to the recently changed XTransformers Oct 6, 2016
public interface FlowableTransformer<Upstream, Downstream> {
Publisher<? extends Downstream> apply(Flowable<Upstream> flowable) throws Exception;
/**
* Applies a function to the upstream flowable and returns a Publisher with
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: uppercase Flowable

@akarnokd
Copy link
Member Author

akarnokd commented Oct 6, 2016

We may have to drop jacoco as it kills builds 1 out of 3.

@codecov-io
Copy link

Current coverage is 81.96% (diff: 100%)

Merging #4674 into 2.x will decrease coverage by 0.13%

@@                2.x      #4674   diff @@
==========================================
  Files           565        565          
  Lines         37414      37414          
  Methods           0          0          
  Messages          0          0          
  Branches       5746       5746          
==========================================
- Hits          30715      30666    -49   
- Misses         4619       4658    +39   
- Partials       2080       2090    +10   

Powered by Codecov. Last update c68077b...34f05a1

@akarnokd akarnokd merged commit 4562e69 into ReactiveX:2.x Oct 6, 2016
@akarnokd akarnokd deleted the TransformerCleanup branch October 6, 2016 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants