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

Pass a shell class instead of a string to xonsh.main.setup. #10

Merged
merged 2 commits into from
Mar 16, 2023

Conversation

joouha
Copy link
Contributor

@joouha joouha commented Mar 16, 2023

Fixes #6

The jupyter kernel was failing to start as xonsh.shell.Shell no longer accepted "jupyter" as a known shell name, and instead required a shell class to be passed.

The jupyter kernel was failing to start as `xonsh.shell.Shell` no longer
accepted "jupyter" as a known shell name, and instead required a shell class to
be passed.

Fixes xonsh#6
@anki-code
Copy link
Member

anki-code commented Mar 16, 2023

Thanks @joouha ! Welcome!

@anki-code anki-code merged commit f6bdeb3 into xonsh:main Mar 16, 2023
@anki-code
Copy link
Member

@joouha please give this repo a star to support this integration. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error starting xonsh kernel: xonsh.__amalgam__.XonshError: jupyter is not recognized as a shell type
2 participants