Skip to content

fix #104698 Make query element of uri survive conflict resolution massaging#104699

Merged
bpasero merged 5 commits intomicrosoft:masterfrom
gjsjohnmurray:fix-104698
Aug 17, 2020
Merged

fix #104698 Make query element of uri survive conflict resolution massaging#104699
bpasero merged 5 commits intomicrosoft:masterfrom
gjsjohnmurray:fix-104698

Conversation

@gjsjohnmurray
Copy link
Copy Markdown
Contributor

This PR fixes #104698. The fixed code originated with @bpasero

@bpasero bpasero added this to the July 2020 milestone Aug 17, 2020
@bpasero bpasero assigned bpasero and unassigned chrmarti Aug 17, 2020
@bpasero bpasero modified the milestones: July 2020, August 2020 Aug 17, 2020
Copy link
Copy Markdown
Member

@bpasero bpasero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Slight change suggested, otherwise looks good to me.

@bpasero bpasero merged commit 04f2a74 into microsoft:master Aug 17, 2020
@bpasero
Copy link
Copy Markdown
Member

bpasero commented Aug 17, 2020

Thanks 👍

bpasero added a commit that referenced this pull request Aug 17, 2020
@gjsjohnmurray gjsjohnmurray deleted the fix-104698 branch August 17, 2020 15:44
gjsjohnmurray added a commit to gjsjohnmurray/vscode that referenced this pull request Aug 17, 2020
…ution massaging (microsoft#104699)

* fix microsoft#104698 Make query element of uri survive conflict resolution massaging

* Incorporate PR feedback

* Simplify

* Event simpler, and with comments

* Variant preferred by @bpasero
gjsjohnmurray pushed a commit to gjsjohnmurray/vscode that referenced this pull request Aug 17, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Oct 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FileSystemProvider whose uris use the query element is not properly handled by conflict resolution feature

3 participants