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

1.x: update sample(time) diagram to indicate emission of last #5007

Merged
merged 1 commit into from
Jan 24, 2017

Conversation

akarnokd
Copy link
Member

This PR updates the 1.x sample(time) diagram to indicate the last value is emitted when the sequence completes. It is the same (style) as for 2.x:

Default scheduler:
image

Custom scheduler:
image

Related: #4952

@codecov-io
Copy link

codecov-io commented Jan 19, 2017

Current coverage is 84.21% (diff: 100%)

Merging #5007 into 1.x will increase coverage by 0.01%

@@                1.x      #5007   diff @@
==========================================
  Files           288        288          
  Lines         17806      17806          
  Methods           0          0          
  Messages          0          0          
  Branches       2698       2698          
==========================================
+ Hits          14992      14995     +3   
+ Misses         1961       1957     -4   
- Partials        853        854     +1   

Powered by Codecov. Last update 9f02563...0a353d3

Copy link
Contributor

@ZacSweers ZacSweers left a comment

Choose a reason for hiding this comment

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

LGTM

@akarnokd akarnokd merged commit 56d94b2 into ReactiveX:1.x Jan 24, 2017
@akarnokd akarnokd deleted the SampleJavadoc1x branch January 24, 2017 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants