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: Schedulers.from() to call RxJavaHooks.onScheduleAction #4311

Merged
merged 1 commit into from
Aug 8, 2016

Conversation

akarnokd
Copy link
Member

@akarnokd akarnokd commented Aug 8, 2016

The ExecutorScheduler wrapper did not call the RxJavaHooks.onScheduledAction() to decorate actions sent to a custom scheduler.

@akarnokd akarnokd added the Bug label Aug 8, 2016
@codecov-io
Copy link

Current coverage is 84.43% (diff: 100%)

Merging #4311 into 1.x will decrease coverage by 0.06%

@@                1.x      #4311   diff @@
==========================================
  Files           268        268          
  Lines         17476      17478     +2   
  Methods           0          0          
  Messages          0          0          
  Branches       2664       2664          
==========================================
- Hits          14766      14757     -9   
- Misses         1851       1863    +12   
+ Partials        859        858     -1   

Powered by Codecov. Last update 27c782d...970a609

@JakeWharton
Copy link
Contributor

👍

@akarnokd akarnokd merged commit b3a0446 into ReactiveX:1.x Aug 8, 2016
@akarnokd akarnokd deleted the CustomSchedulerHookFix branch August 8, 2016 13:30
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.

3 participants