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

[git] merge operation without modified files cannot be completed #7187

Open
federicobozzini opened this issue Feb 20, 2020 · 1 comment
Open
Labels
bug bugs found in the application git issues related to git help wanted issues meant to be picked up, require help

Comments

@federicobozzini
Copy link
Contributor

federicobozzini commented Feb 20, 2020

Description

If a merge operation doesn't result in some file changes, it is not possible to commit. This makes impossible to complete the merge operation without using the terminal. The message shown to the user is No changes added to commit.

There is also no visual clue to the user to signal that there is an ongoing merge in this situation.

Reproduction Steps

The simplest case is the one where the user performing a merge operation sees only a conflicted file and nothing else. If none of the incoming changes is accepted, the file is not shown as modified and so it cannot be commited. But committing is necessary in this case to complete the merge operation (the git status command shows All conflicts fixed but you are still merging. (use "git commit" to conclude merge)).

If the reproduction steps are not clear I can create a repository to show the issue.

OS and Theia version:

Seen on the master version of Theia.

Diagnostics:

@federicobozzini federicobozzini changed the title [git] merge operation without modified files cannot be commited [git] merge operation without modified files cannot be completed Feb 20, 2020
@akosyakov akosyakov added bug bugs found in the application git issues related to git help wanted issues meant to be picked up, require help labels Feb 20, 2020
@westbury
Copy link
Contributor

I decided to check this out using the vscode Git builtin instead of the Theia Git extension. Surprisingly it's an issue there too. See microsoft/vscode#7989. That's been open over three years.

We can fix this but only when using Theia Git extension.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application git issues related to git help wanted issues meant to be picked up, require help
Projects
None yet
Development

No branches or pull requests

3 participants