Skip to content

Conversation

@mAu888
Copy link
Contributor

@mAu888 mAu888 commented Feb 21, 2017

What's in here

This removes the unused RxBlocking dependency in the Action target. We found that issue while including it into our project and it suddenly required RxBlocking as well. Looking at the code there should actually be no need for it. Tests and Demo app run without issue with the library removed.

Copy link
Member

@ashfurrow ashfurrow left a comment

Choose a reason for hiding this comment

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

Looks good to me, not sure why that was in there 👍

@ashfurrow ashfurrow merged commit 3af9414 into RxSwiftCommunity:master Feb 21, 2017
@ashfurrow
Copy link
Member

Thanks a lot for contributing! I've invited you to join the RxSwiftCommunity GitHub organization – no pressure to accept! If you'd like more information on what that means, check out our contributor guidelines and feel free to reach out with any questions.

@mAu888 mAu888 deleted the remove_rxblocking branch February 22, 2017 09:27
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.

2 participants