Skip to content
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

Use channel get_msg helper method #634

Merged
merged 1 commit into from
Apr 12, 2021

Conversation

davidbrochart
Copy link
Collaborator

Use e.g. kc.get_shell_msg() instead of kc.shell_channel.get_msg(), because the latter is now always async in jupyter_client. The former is a higher-level method that runs get_msg() until complete for a blocking client.

@davidbrochart
Copy link
Collaborator Author

The failing tests timed out, and they are using jupyter_client 6.1.12, so I am not sure what is the reason.

@davidbrochart
Copy link
Collaborator Author

Trigger CI

@davidbrochart davidbrochart marked this pull request as ready for review April 12, 2021 20:15
@davidbrochart
Copy link
Collaborator Author

This is now ready to review/merge.

@blink1073 blink1073 added this to the 6.0 milestone Apr 12, 2021
Copy link
Contributor

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@blink1073 blink1073 merged commit 5d21c8f into ipython:master Apr 12, 2021
tschaume added a commit to tschaume/kernel_gateway that referenced this pull request Aug 21, 2021
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Oct 25, 2021
https://build.opensuse.org/request/show/925994
by user bnavigator + dimstar_suse
- Allow updating to jupyter_client 7 through backporting
  gh#ipython/ipykernel#634 and gh#ipython/ipykernel#736 to
  ipykernel-pr634+736-remove-block-client7.patch
- Update to 5.5.6
  * Add ipython_genutils dependency
- Block updating to jupyter_client 7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants