Skip to content

Conversation

@WillSewell
Copy link
Contributor

@WillSewell WillSewell commented Aug 9, 2019

Description of the pull request

In 569b765, we switched from using https://github.com/TooTallNate/Java-WebSocket to a fork https://github.com/pusher/java-websocket because the original was not actively maintained.

I've manually tested the change in an Android app.

Why is the change necessary?

https://github.com/TooTallNate/Java-WebSocket is now actively maintained again. We'd rather not maintain a fork.


CC @pusher/mobile

Copy link

@damdo damdo left a comment

Choose a reason for hiding this comment

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

LGTM

@WillSewell WillSewell merged commit 90d4ee7 into master Aug 9, 2019
@WillSewell WillSewell deleted the non-forked-websocket-library branch August 9, 2019 11:20
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.

3 participants