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
[MRG] Add hooks for custom server implementations #453
Conversation
9d876d8
to
dca3864
Compare
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.
Needs:
- Change to lowercase_underscore variable/parameter names.
- Unit testing for the new feature (even if its just a nominal test)
- Documentation
- Add
versionchanged
andversionadded
directives as appropriate - Add missing docstring
- Add change to release notes
- Add minimal explanation of the requirements of the
request_handler
to the docstring (i.e.BaseRequestHandler
subclass,handle
method), and/or point to the Python docs.
- Add
dca3864
to
d4dc344
Compare
OK, I think this is ready to go. Let me know if there are further changes to make. Also, feel free to push to my branch with any caretaker changes. |
I think the coverage failure here is flake--all new code is covered. |
Made a slight tweak to make |
Very nice. Thanks! |
Thanks for merging EDIT: woops, 1.5.0 not released yet |
Reference issue
#452
Tasks