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
deprecate IPYTHON_DIR in favor of IPYTHONDIR #1693
Conversation
@@ -260,7 +260,10 @@ def get_ipython_dir(): | |||
xdg_dir = get_xdg_dir() | |||
|
|||
# import pdb; pdb.set_trace() # dbg | |||
ipdir = env.get('IPYTHON_DIR', env.get('IPYTHONDIR', None)) | |||
if 'IPYTHON_DIR' in env: | |||
warnings.warn('The environment variable IPYTHON_DIR is depreciated. ' |
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.
s/depreciated/deprecated/
Looks sensible to me, after fixing the little typos that we are deprecating the variable, not depreciating it. |
Spelling fixed (force push, sorry), and a new commit describing the history of the variable. |
Thanks! And no worries about force push - it's always okay to force-push to a PR, I do it to fix spelling errors or rewrite commit messages all the time. |
deprecate IPYTHON_DIR in favor of IPYTHONDIR IPYTHON_DIR was a proposed rename, whose added complexity does not outweigh any potential benefit, and is reverted. closes #1167
deprecate IPYTHON_DIR in favor of IPYTHONDIR IPYTHON_DIR was a proposed rename, whose added complexity does not outweigh any potential benefit, and is reverted. closes ipython#1167
Closes #1167.