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 unused declarations #4527

Merged
merged 1 commit into from
Sep 11, 2016
Merged

2.x: Remove unused declarations #4527

merged 1 commit into from
Sep 11, 2016

Conversation

vanniktech
Copy link
Collaborator

No description provided.

@codecov-io
Copy link

codecov-io commented Sep 11, 2016

Current coverage is 78.78% (diff: 100%)

Merging #4527 into 2.x will increase coverage by 0.05%

@@                2.x      #4527   diff @@
==========================================
  Files           515        515          
  Lines         34628      34615    -13   
  Methods           0          0          
  Messages          0          0          
  Branches       5419       5418     -1   
==========================================
+ Hits          27263      27272     +9   
+ Misses         5399       5390     -9   
+ Partials       1966       1953    -13   

Powered by Codecov. Last update ca4bba6...4addc15

@akarnokd akarnokd added this to the 2.0 RC 3 milestone Sep 11, 2016
@akarnokd akarnokd merged commit 9a585a7 into ReactiveX:2.x Sep 11, 2016
@vanniktech vanniktech deleted the 2.x_unused branch September 11, 2016 14:16
@akarnokd
Copy link
Member

Thanks! FYI, I'm adding operators to Maybe so please avoid that class and related components until I post the PR.

@vanniktech
Copy link
Collaborator Author

Alright didn't plan creating any other PR today. Tomorrow I have a few things planned though.

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