Skip to content

nbconvert: Add cwd to sys.path #3752

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

Merged
merged 3 commits into from
Jul 26, 2013
Merged

Conversation

jdfreder
Copy link
Contributor

This makes transformers and filters in the cwd load
from dotted object names.

Disclaimer: The new init def is not required, I thought it looked better (so the initialize def only contains init... functions). If we remove it the changes are one line_

jdfreder added 2 commits July 23, 2013 18:52
This makes transformers and filters in the cwd load
from dotted object names.
@ivanov
Copy link
Member

ivanov commented Jul 24, 2013

are there any relative imports in nbconvert code path that might be affected by this?

"""
Add the cwd to the sys.path ($PYTHONPATH)
"""
sys.path.append(os.getcwd())
Copy link
Member

Choose a reason for hiding this comment

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

typically you put this at the front (path.insert(0, '.')), so it's highest priority.

@jdfreder
Copy link
Contributor Author

@ivanov not that I know of
@minrk ok ill change that when I wake up

@jdfreder
Copy link
Contributor Author

done

@jdfreder
Copy link
Contributor Author

bump

Carreau added a commit that referenced this pull request Jul 26, 2013
nbconvert: Add cwd to sys.path

This makes transformers and filters in the cwd load
from dotted object names.
@Carreau Carreau merged commit 9df1a04 into ipython:master Jul 26, 2013
@jdfreder jdfreder deleted the add_cwd2path branch March 10, 2014 18:41
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
nbconvert: Add cwd to sys.path

This makes transformers and filters in the cwd load
from dotted object names.
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.

4 participants