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

Introduce Mono.delayElement, prepare move to Flux.delayElements #373

Merged
merged 8 commits into from Jan 18, 2017

Conversation

simonbasle
Copy link
Member

Flux.delay to be renamed delayElements. For consistency, adds a Mono.delayElement instance method.

This includes a first pass at the new MonoDelayElement operator.

@codecov-io
Copy link

codecov-io commented Jan 18, 2017

Current coverage is 67.08% (diff: 98.18%)

Merging #373 into master will increase coverage by 0.07%

@@             master       #373   diff @@
==========================================
  Files           260        261     +1   
  Lines         23730      23783    +53   
  Methods           0          0          
  Messages          0          0          
  Branches       4335       4342     +7   
==========================================
+ Hits          15903      15956    +53   
- Misses         6119       6123     +4   
+ Partials       1708       1704     -4   

Powered by Codecov. Last update b4d03b5...18f04b9

@simonbasle simonbasle force-pushed the 263-delayToDelayEachDeprecation branch from af5891f to b9e527d Compare January 18, 2017 10:49
@simonbasle
Copy link
Member Author

simonbasle commented Jan 18, 2017

(just rebased on master, will rework the tests to increase coverage)

smaldini
smaldini previously approved these changes Jan 18, 2017
@simonbasle simonbasle dismissed smaldini’s stale review January 18, 2017 15:30

need re-review, especially javadoc wording

@simonbasle simonbasle merged commit 74c904c into master Jan 18, 2017
@simonbasle simonbasle deleted the 263-delayToDelayEachDeprecation branch January 18, 2017 15:59
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.

None yet

3 participants