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

Fix #11527 initialize posix aliases after a %reset #11528

Merged
merged 1 commit into from Jan 6, 2019
Merged

Conversation

@jwidner
Copy link
Contributor

@jwidner jwidner commented Dec 12, 2018

Changes the behavior of %reset by initializing the posix aliases clear, less, more, and man during a reset. It's not clear to me what the intended behavior should be, but it fixes #11527.

For example, running %reset in IPython, and then trying to run
%clear raises a UsageError.
@Carreau Carreau added this to the 7.3 milestone Jan 6, 2019
@Carreau
Copy link
Member

@Carreau Carreau commented Jan 6, 2019

That seem reasonable to me. Thanks.

Loading

@Carreau Carreau merged commit e9f3be2 into ipython:master Jan 6, 2019
4 checks passed
Loading
@red8012
Copy link

@red8012 red8012 commented Mar 10, 2019

It seems that the change causes the following message being printed whenever a reset is done.

ERROR:root:Invalid alias: The name clear can't be aliased because it is another magic command.
ERROR:root:Invalid alias: The name more can't be aliased because it is another magic command.
ERROR:root:Invalid alias: The name less can't be aliased because it is another magic command.
ERROR:root:Invalid alias: The name man can't be aliased because it is another magic command.

See https://stackoverflow.com/questions/54971047/spyder-reports-invalid-alias-when-running-any-script. The issue happens not only on Spyder but also on Jupyter notebook.

Loading

@Carreau
Copy link
Member

@Carreau Carreau commented Mar 12, 2019

Can you please open a new issue for this ? Thanks.

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants