Skip to content

Revert "Don't complete quotes if they complete a valid string (#60227)"#60851

Merged
CyrusNajmabadi merged 1 commit intodotnet:release/dev17.2from
CyrusNajmabadi:revert60227
Apr 20, 2022
Merged

Revert "Don't complete quotes if they complete a valid string (#60227)"#60851
CyrusNajmabadi merged 1 commit intodotnet:release/dev17.2from
CyrusNajmabadi:revert60227

Conversation

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

@CyrusNajmabadi CyrusNajmabadi commented Apr 20, 2022

Fixes AB#1525830

This reverts commit a838dcf.

That PR (AB#1525830) introduced a hang. We will reintroduce this back in 17.3 with the hang fixed.

@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner April 20, 2022 00:01
@ghost ghost added the Area-IDE label Apr 20, 2022
@jinujoseph
Copy link
Copy Markdown
Contributor

cc @davidwengier for info

@CyrusNajmabadi CyrusNajmabadi requested review from a team and dibarbet April 20, 2022 00:03
@davidwengier
Copy link
Copy Markdown
Member

Whoops! Thanks

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor Author

Preemptively merging after convo with @jinujoseph .

@CyrusNajmabadi CyrusNajmabadi merged commit 1e40aa1 into dotnet:release/dev17.2 Apr 20, 2022
@CyrusNajmabadi CyrusNajmabadi deleted the revert60227 branch April 20, 2022 18:37
dibarbet added a commit that referenced this pull request Apr 21, 2022
This reverts commit 1e40aa1, reversing
changes made to f2d278f.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants