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
Gets us to 100% test coverage on wsproto.frame_protocol. #29
Conversation
Sweeeeeet flake8 is complaining for some reason |
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.
Looks good to me.
I noticed that parse_more_gen
still has some stray code to filter enabled extensions, which is now redundant. Could make it a separate PR, but if I did it now it'd probably conflict with this so mentioning it and you can do it or hopefully I'll remember :-)
Codecov Report
@@ Coverage Diff @@
## master #29 +/- ##
==========================================
+ Coverage 86.06% 87.98% +1.91%
==========================================
Files 7 7
Lines 847 849 +2
Branches 186 184 -2
==========================================
+ Hits 729 747 +18
+ Misses 76 68 -8
+ Partials 42 34 -8
Continue to review full report at Codecov.
|
Fixes #25, addresses #26.