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

Merge Editor: Detect/resolve conflict by adding same content in both inputs #158908

Closed
lszomoru opened this issue Aug 23, 2022 · 2 comments
Closed
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) merge-editor

Comments

@lszomoru
Copy link
Member

Testing #158819

  • Clone https://github.com/hediet/diff-merge-dataset
  • Use the Load Merge Editor State From Folder command and select the merges/demo1 folder
  • Select result in the quick pick
  • The first conflict is caused by the same content being added in both inputs on line 44. Should this edge case be detected and automatically resolved by accepting either one of the two sides?
    image
@hediet
Copy link
Member

hediet commented Aug 24, 2022

/duplicate #158942 and #157496

@vscodenpa vscodenpa added the *duplicate Issue identified as a duplicate of another issue(s) label Aug 24, 2022
@vscodenpa
Copy link

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.

Happy Coding!

@vscodenpa vscodenpa closed this as not planned Won't fix, can't repro, duplicate, stale Aug 24, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Oct 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) merge-editor
Projects
None yet
Development

No branches or pull requests

3 participants