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

Disable option for rebase interactive editor is not honoured #1687

Closed
richard-swyftx opened this issue Oct 14, 2021 · 22 comments
Closed

Disable option for rebase interactive editor is not honoured #1687

richard-swyftx opened this issue Oct 14, 2021 · 22 comments
Assignees
Labels
blocked: vscode Caused by or dependent on a VS Code issue or feature blocked Blocked or caused by an upstream issue or feature bug Something isn't working
Milestone

Comments

@richard-swyftx
Copy link

richard-swyftx commented Oct 14, 2021

The setting "git-rebase-todo": "default" is not honoured

  • GitLens Version: 11.6.1
  • Git Version: 2.25.1
  • VSCode Version: 1.61.0
  • OS Version: Linux 5.11.0-37-generic, 20.04.2-Ubuntu

NOTE: Perform a git rebase interactive == terminal/command-line: git rebase -i origin/dev

Steps to Reproduce:

Option 1)

  1. Perform a git rebase interactive
  2. Select the "Disable rebase editor" -> This updates the VSCode settings file (like option 2)
  3. Perform a git rebase interactive - Still displays the UI editor, instead of the text editor

Option 2)

  1. Edit the VSCode settings file and add:
  "workbench.editorAssociations": {
    "git-rebase-todo": "default"
  }
  1. Perform a git rebase interactive - Still displays the UI editor, instead of the text editor
@richard-swyftx richard-swyftx added potential-bug triage Needs to be looked at labels Oct 14, 2021
@eamodio
Copy link
Member

eamodio commented Nov 4, 2021

@richard-swyftx I can't see to reproduce this -- How are you performing a rebase? Directly from the terminal?

@eamodio eamodio added needs-more-info Needs further information, steps, details, etc. and removed triage Needs to be looked at labels Nov 4, 2021
@samlh
Copy link

samlh commented Nov 4, 2021

I was hitting this too - it happens to me when rebasing from the command line, with VS Code not already open.

@richard-swyftx
Copy link
Author

richard-swyftx commented Nov 5, 2021

@eamodio it happens when I rebase from the command line (I've updated the issue description to reflect).

@github-actions
Copy link

This issue needs more information and has not had recent activity. Please provide the missing information or it will be closed in 7 days. Thanks!

@github-actions github-actions bot added the inactive Issue has not had recent required feedback label Nov 12, 2021
@gardenvarietyse
Copy link

this started happening for me too recently when I rebase from terminal, for example git rebase -i HEAD~2, with interactive editor disabled in settings

vscode is invoked like so:

$ echo $EDITOR
/usr/local/bin/code -n -w

vscode version: 1.62.1 on macbook pro m1
gitlens: 11.6.1
git: 2.30.1 (Apple Git-130)

@andrewnystrom
Copy link

Same issue for me as well. Happy to help diagnose.
Visual Studio Code version: Version: 1.62.1
MacOS: 12.0.1
GitLense: v11.6.1

@github-actions github-actions bot removed the inactive Issue has not had recent required feedback label Nov 13, 2021
@rivy
Copy link
Contributor

rivy commented Nov 13, 2021

Same problem from the Windows command line...

Windows: v10.0.19042.1348
VSC version: 1.62.2 , 3a6960b964327f0e3882ce18fcebd07ed191b316 , x64
GitLens: v11.6.1

And being forced into the UI is breaking some more complex rebase chains.
I will have to disable GitLens until this is resolved.

From some minimal testing, it looks like reversion to VSCode v1.59.1 fixes the issue.

@medifle
Copy link

medifle commented Nov 18, 2021

Same issue.

[email protected]
[email protected]
[email protected]
[email protected]

With a VSC workspace opened and Interactive Rebase Editor disabled in GitLens Settings, I tried git rebase -i <commit> on a feature branch and got the UI editor instead of text editor.

It is 100% reproducible on my side.

@eamodio
Copy link
Member

eamodio commented Nov 19, 2021

I'm really sorry that a lot of you are experiencing this issue, but unfortunately I can't seem to reproduce it or see how to fix it. VS Code doesn't provide much for extension when registering this type of custom editor (the rebase UI is a custom editor based on the git-rebase-todo file). The only way VS Code allows an extension to change the custom editor usage is through the workbench.editorAssociations setting.

With the setting set as follows -- the GitLens Interactive Rebase Editor should not be used. At least for when it gets invoked from a terminal -- if you start an interactive rebase from GitLens, via content menu or Git Command Palette, it will still default to the UI.

  "workbench.editorAssociations": {
    "git-rebase-todo": "default"
  }

If the setting is missing or there is no entry for "git-rebase-todo" or "git-rebase-todo": "gitlens.rebase" then the GitLens UI will be used.

Can you all verify that the setting is set appropriately? Thanks for any help you all can provide.

@eamodio
Copy link
Member

eamodio commented Nov 19, 2021

@mjbvz Do you have any ideas on how I could debug this?

@rivy
Copy link
Contributor

rivy commented Nov 20, 2021

Cross-post as a bug to vscode?

@PiterGoCompany
Copy link

Same issue:
Windows 11 Pro 21H2 22000.318
Visual Studio Code: 1.62.3
GitLens: 11.7.0
GIT: 2.34.0.windows.1

@eamodio
Copy link
Member

eamodio commented Nov 22, 2021

Cross-posted: microsoft/vscode#137675

@eamodio
Copy link
Member

eamodio commented Nov 23, 2021

It looks like this is a VS Code bug -- can any of you confirm that this only happens when there is no other VS Code window already open?

@eamodio eamodio added blocked Blocked or caused by an upstream issue or feature blocked: vscode Caused by or dependent on a VS Code issue or feature bug Something isn't working and removed needs-more-info Needs further information, steps, details, etc. potential-bug labels Nov 23, 2021
@eamodio eamodio added this to the Soon™ milestone Nov 23, 2021
@eamodio eamodio self-assigned this Nov 23, 2021
@thec0keman
Copy link

This happens to me frequently when there is another VS Code window open.

@PiterGoCompany
Copy link

@eamodio indeed it happens when there is no other VS Code window.

@rivy
Copy link
Contributor

rivy commented Dec 21, 2021

@eamodio , thanks for looking into this and creating an issue on the VSCode repo.

As this breaks my more complicated rebases, I'll have to hold at VSCode v1.59.1 until this is fixed.

@eamodio
Copy link
Member

eamodio commented Dec 21, 2021

I'm really sorry, and am hoping VS Code will fix this very soon. Other than removing the interactive rebase completely, there isn't anything that I know of that GitLens can do to help this (because of the static registration VS Code requires for custom editors).

At the same time, @rivy can you tell me (or file a new issue ideally) to what complicated rebases that the interactive rebase editor breaks?

@torokati44
Copy link

Other than removing the interactive rebase completely

I really don't mean to sound entitled or rude, but this is about the third time this feature (which I have never asked for) causes me annoyance and inconvenience, and this will be the third time that I will try really hard to disable it. So, I wouldn't miss it. :/

@eamodio
Copy link
Member

eamodio commented Jan 8, 2022

@torokati44 Believe me I really sympathize and am frustrated by this situation as well. Please up vote and add your feed back to the VS Code bug that is causing this: microsoft/vscode#137675.

I am going to try out a possible workaround (mentioned in that issue), that might avoid the issue, but it's not a great long term solution -- as I really don't like changing user preferences without their input.

@lramos15
Copy link

This has now been fixed upstream microsoft/vscode#137675. Feel free to test it out in the latest insiders https://code.visualstudio.com/insiders/

@eamodio eamodio closed this as completed Feb 28, 2022
@eamodio eamodio added pending-release Resolved but not yet released to the stable edition and removed pending-release Resolved but not yet released to the stable edition labels Mar 1, 2022
@eamodio eamodio modified the milestones: Soon™, Shipped Mar 1, 2022
@github-actions
Copy link

github-actions bot commented Apr 1, 2022

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
blocked: vscode Caused by or dependent on a VS Code issue or feature blocked Blocked or caused by an upstream issue or feature bug Something isn't working
Projects
None yet
Development

No branches or pull requests

12 participants