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
Better error type #82
Conversation
cf237d0
to
ef82bd6
Compare
Codecov Report
@@ Coverage Diff @@
## master #82 +/- ##
==========================================
- Coverage 55.36% 54.21% -1.16%
==========================================
Files 11 11
Lines 587 629 +42
==========================================
+ Hits 325 341 +16
- Misses 262 288 +26
Continue to review full report at Codecov.
|
Epic work! But sorry I'm still got busy in work. Will review it in next holiday. |
No hurry, take your time! |
ef82bd6
to
decb6de
Compare
rebased with latest master |
Merged, thank for contributions! @letmutx |
Fixes #79
I may have gone a bit overboard refactoring the binary protocol. Sorry about that!
The issue with returning same errors in both ASCII and binary protocol is that the format of errors is different. We should instead document that the errors returned by both the protocols may be different but guarantee that the same error is returned in the future versions too for compatibility.