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

Add TermSocket back from nbclassic #314

Merged
merged 2 commits into from
Sep 29, 2020

Conversation

jasongrout
Copy link
Contributor

#313 removed too much to nbclassic. Reverting part of it allows JupyterLab to work with the standalone jupyter server (and JupyterLab examples to work without loading other extensions like nbclassic).

jupyter-server#313 removed too much to nbclassic. Reverting part of it allows JupyterLab to work with the standalone jupyter server (and JupyterLab examples to work without loading other extensions like nbclassic).
jasongrout added a commit to jasongrout/nbclassic that referenced this pull request Sep 29, 2020
jupyter#31 moved a bit too much from jupyter server to nbclassic. This moves some back. See also jupyter-server/jupyter_server#314
@jasongrout jasongrout merged commit d1c34f4 into jupyter-server:master Sep 29, 2020
@kevin-bates
Copy link
Member

I'm actually glad to see this compensating pull request as we would lose the ability to cull terminals once the corresponding notebook PR was ported. Thanks.

@jasongrout
Copy link
Contributor Author

Great!

@jasongrout
Copy link
Contributor Author

If it's all right with you, I'll push out a release with this.

@kevin-bates
Copy link
Member

Totally fine Jason - thanks.

@jasongrout
Copy link
Contributor Author

This is now published as 1.0.3

Zsailer added a commit to Zsailer/jupyter_server that referenced this pull request Nov 18, 2022
hMED22 pushed a commit to hMED22/jupyter_server that referenced this pull request Jan 23, 2023
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.

2 participants