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

2.x: Remove checked exceptions from transformer interfaces. #4710

Merged
merged 1 commit into from
Oct 19, 2016
Merged

2.x: Remove checked exceptions from transformer interfaces. #4710

merged 1 commit into from
Oct 19, 2016

Conversation

JakeWharton
Copy link
Contributor

These functions are for transforming the stream shape, not doing work. Any operation that would throw a checked exception should happen inside the stream, not when shaping it.

Closes #4709.

These functions are for transforming the stream shape, not doing work. Any operation that would throw a checked exception should happen inside the stream, not when shaping it.
@codecov-io
Copy link

codecov-io commented Oct 15, 2016

Current coverage is 87.14% (diff: 100%)

Merging #4710 into 2.x will decrease coverage by 0.04%

@@                2.x      #4710   diff @@
==========================================
  Files           568        568          
  Lines         37061      37046    -15   
  Methods           0          0          
  Messages          0          0          
  Branches       5648       5648          
==========================================
- Hits          32312      32283    -29   
- Misses         3050       3062    +12   
- Partials       1699       1701     +2   

Powered by Codecov. Last update 637978c...b95c0d4

@akarnokd akarnokd added this to the 2.0 RC5 milestone Oct 15, 2016
@akarnokd akarnokd merged commit edd8aed into ReactiveX:2.x Oct 19, 2016
@JakeWharton JakeWharton deleted the jw/check-checked-transformers branch October 19, 2016 16:08
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