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

Get rid of the last shim warning with IPython/Jupyter 4 #2741

Merged
merged 2 commits into from
Oct 7, 2015
Merged

Get rid of the last shim warning with IPython/Jupyter 4 #2741

merged 2 commits into from
Oct 7, 2015

Conversation

stonebig
Copy link
Contributor

@stonebig stonebig commented Oct 4, 2015

No description provided.

try: # Ipython 4
import nbformat as nbformat
from nbconvert import PythonExporter as nbexporter
except: # Ipython 3
Copy link
Member

Choose a reason for hiding this comment

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

Thanks for this @stonebig

But we should catch the actual exception, like except ImportError?

@ccordoba12
Copy link
Member

+1 to the changes suggested by @goanpeca and @Nodd. After that, this is ready to be merged.

@ccordoba12
Copy link
Member

Thanks!

@ccordoba12 ccordoba12 changed the title get rid of shim warning with Ipython 4 Get rid of the last shim warning with IPython/Jupyter 4 Oct 7, 2015
ccordoba12 added a commit that referenced this pull request Oct 7, 2015
Get rid of the last shim warning with IPython/Jupyter 4
@ccordoba12 ccordoba12 merged commit 07af7fd into spyder-ide:master Oct 7, 2015
@stonebig
Copy link
Contributor Author

stonebig commented Oct 7, 2015

You're welcomed.

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.

4 participants