-
Notifications
You must be signed in to change notification settings - Fork 28.9k
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
staging file after resolving merge conflicts does not warn if file is not saved #34175
Comments
my issue appears to be related to this one. After a git merge, I go through the process of resolving the merge conflicts on an individual file. I then save the document by using control + s. Then when I click the Add button to commit, I get an error message saying there are merge conflicts. This screencast demonstrates what I see. |
I have the exact same issue as @minoseah629 As soon as I resolve a conflict and save the file, It gives me a warning about saving with merge conflicts. |
I think @minoseah629 's issue is separate and definitely needs to be created as an issue, as it is literally a bug. This issue I also strongly agree with but is more of a feature request. I frequently commit files with merge conflicts in them because of this UX issue. |
@peabnuts123 I'm more than willing to create a separate issue for this, but it's not a feature, it's an actual bug ;) Even when all merge conflicts were resolved, it still shows that message. |
Yes, somehow recently the behavior changed from |
I know it's a bug… that's why it should be an issue. I said it was a bug. However, it is not related to this issue, and should be tracked in a separate one. |
So with the v 1.18.0, my issue of the merge conflict has been resolved. I am not sure if this original issue was resolved. |
I believe your issue is somewhat different than this issue. I would suggest making a new issue, but thats just my opinion. And since this issue is closed, microsoft would mostly likely need a new issue to work towards for a possible fix. |
Yes. This issues appears to be resolved now. At least for me. I get a warning if I didn't save first. Staging just works as expected with no false warning when the merge conflicts are resolved and saved. |
This issue was resolved as a part of a recent update. It's kind of unintelligent as it warns for any unsaved files in your workspace (as opposed to just ones you are staging) but it gets the job done, and I appreciate the warning. This issue can be closed. |
(1 theme extensions excluded)
Steps to Reproduce:
Expected result:
resolved file is staged
Actual result:
The resolved file was in an unsaved state so the unresolved file was staged without any warning.
The text was updated successfully, but these errors were encountered: