-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
WebSocketServer code quality #789
Conversation
Reduce complexity for WebSocketServer
For WebSocketFactory and Compression Extension
@PhilipRoman Conflict is not relevant FYI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow this is a lot easier to read now! 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow this is a lot easier to read now! 👍
GitHub is not letting me "review" at the moment so I'm leaving this here. The changes are awesome! I didn't check all the tests but the ones I checked are good. 👍 |
1 similar comment
GitHub is not letting me "review" at the moment so I'm leaving this here. The changes are awesome! I didn't check all the tests but the ones I checked are good. 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow this is a lot easier to read now! 👍
😆 I did not expect that to happen |
Description
Refactoring as part of improving the code quality
Related Issue
Part of #749
Motivation and Context
Code quality helps fixing issues!
How Has This Been Tested?
All tests pass
Types of changes
Checklist: