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

Remove set-next input when triggering help. #13625

Merged
merged 1 commit into from Apr 5, 2022
Merged

Conversation

Carreau
Copy link
Member

@Carreau Carreau commented Apr 5, 2022

This was a long standing feature from when the main way to edit code was
readline. Now that we have proper history and editing frontend,
including prompt toolkit this is not necessary, especially since it
creates issue like in JupyterLab.

Should close #13602

@Carreau Carreau force-pushed the nosni branch 2 times, most recently from e9de4cc to 6b06908 Compare April 5, 2022 12:41
This was a long standing feature from when the main way to edit code was
readline. Now that we have proper history and editing frontend,
including prompt toolkit this is not necessary, especially since it
creates issue like in JupyterLab.

Should close ipython#13602
@Carreau Carreau added this to the 7.33 milestone Apr 5, 2022
@Carreau Carreau merged commit 9272c10 into ipython:master Apr 5, 2022
@lumberbot-app
Copy link
Contributor

lumberbot-app bot commented Apr 5, 2022

Owee, I'm MrMeeseeks, Look at me.

There seem to be a conflict, please backport manually. Here are approximate instructions:

  1. Checkout backport branch and update it.
git checkout 7.x
git pull
  1. Cherry pick the first parent branch of the this PR on top of the older branch:
git cherry-pick -x -m1 9272c10da28c98237b9330518bdca7916dca686b
  1. You will likely have some merge/cherry-pick conflict here, fix them and commit:
git commit -am 'Backport PR #13625: Remove set-next input when triggering help.'
  1. Push to a named branch:
git push YOURFORK 7.x:auto-backport-of-pr-13625-on-7.x
  1. Create a PR against branch 7.x, I would have named this PR:

"Backport PR #13625 on branch 7.x (Remove set-next input when triggering help.)"

And apply the correct labels and milestones.

Congratulations — you did some good work! Hopefully your backport PR will be tested by the continuous integration and merged soon!

Remember to remove the Still Needs Manual Backport label once the PR gets merged.

If these instructions are inaccurate, feel free to suggest an improvement.

@lumberbot-app lumberbot-app bot added the Still Needs Manual Backport Added My MrMeeseeks when a backport fails. Help by backporting it, solving conflicts, send PR. label Apr 5, 2022
Carreau added a commit to Carreau/ipython that referenced this pull request Apr 7, 2022
Carreau added a commit to Carreau/ipython that referenced this pull request Apr 7, 2022
Carreau added a commit that referenced this pull request Apr 27, 2022
Backport PR #13625: Remove set-next input when triggering help.
@Carreau Carreau removed the Still Needs Manual Backport Added My MrMeeseeks when a backport fails. Help by backporting it, solving conflicts, send PR. label Apr 29, 2022
@Carreau Carreau deleted the nosni branch August 30, 2022 14:51
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.

Transformer for pinfo line magic incorrectly sets next input when the line contains parentheses
1 participant