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
Added debug messages #6704
Added debug messages #6704
Conversation
Thanks for making a pull request to JupyterLab! To try out this branch on binder, follow this link: |
This looks good to me, modulo the docstrings about API stability @jasongrout raised. Is using a |
…ot part of the public api
I think |
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.
I've added docstrings everywhere warning that these things are experimental.
Can someone review my docstrings? @SylvainCorlay, @afshin, @JohanMabille, etc.?
This looks good to me! |
@afshin - feel free to merge if it also looks good to you. I'll merge tomorrow morning (~8 hours) if you haven't and there are no objections. |
Sweet, thank you! |
🎉 |
Awesome! |
References
This PR partially implements changes described in jupyter/jupyter_client#446.
It completes #6544 and allows to send debug_request messages on the Control channel.
This PR should not be merged until the Jupyter Kernel Protocol has been updated as described in jupyter/jupyter_client#446.
Code changes
requestDebug
)User-facing changes
None
Backwards-incompatible changes
None