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

handle empty but defined SHELL env #5

Merged
merged 1 commit into from
Apr 13, 2015
Merged

Conversation

minrk
Copy link
Member

@minrk minrk commented Apr 12, 2015

Sometimes SHELL can be '', which should be treated the same as undefined.

closes #4

Sometimes SHELL can be '',
which should be treated the same as undefined.
@minrk minrk mentioned this pull request Apr 12, 2015
@ellisonbg
Copy link
Contributor

I think this makes sense as a fix, but I would like to understand why I see this behavior some of the time ...will keep looking...

@ellisonbg
Copy link
Contributor

This looks good, merging.

@takluyver
Copy link
Member

This looks good, actually merging ;-)

takluyver added a commit that referenced this pull request Apr 13, 2015
handle empty but defined SHELL env
@takluyver takluyver merged commit a7bd1bb into jupyter:master Apr 13, 2015
jdfreder referenced this pull request in jdfreder/notebook Apr 23, 2015
Fix js-tests for new default selector logic.
@minrk minrk modified the milestone: 4.0 Dec 1, 2015
@minrk minrk deleted the empty-shell branch December 1, 2015 12:33
minrk pushed a commit that referenced this pull request Feb 9, 2016
use session.deserialize to unpack message for rate limiting
ivanov pushed a commit that referenced this pull request Jan 26, 2017
drag modal dialogs by the header only
gutow pushed a commit to gutow/notebook that referenced this pull request Jul 21, 2019
using quill.on to get into edit mode when cell is selected
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't open a terminal
3 participants