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
Split IPython specific functions out of utils.path #8214
Conversation
e70b90e
to
58803c7
Compare
+1 |
@@ -0,0 +1,246 @@ | |||
from contextlib import contextmanager |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume this isn't meant to be in magics
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it wasn't. Fixed.
# will raise ProfileDirError if no such profile | ||
raise IOError("Profile %r not found") | ||
return filefind(filename, ['.', pd.security_dir]) | ||
warn("get_security_file has moved to the IPython.paths module") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
get_security_file is going to leave IPython entirely, since it's a Jupyter function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found we were only using this in one example, and find_connection_file
, which is already in jupyter_client, works as well for that, so I have removed this function entirely.
Split IPython specific functions out of utils.path
Functions like
get_ipython_dir()
are moved to a newIPython.paths
module. This should facilitate moving the remainder of utils.path into genutils. Wrappers are left in utils.path which will still work, but issue a deprecation warning on call.While doing this, I noticed that the code to check for pre-0.11 config files and warn about them is still in place. I think that has served its purpose and can be retired now.