Skip to content

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 reopened this Apr 12, 2021
@davidbrochart davidbrochart marked this pull request as draft April 12, 2021 19:21
@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