Skip to content

[Port] Gracefully handle additional locations in DiagnosticData.Create#65532

Merged
arunchndr merged 1 commit intodotnet:release/dev17.4from
mavasani:PortFix1676229
Nov 21, 2022
Merged

[Port] Gracefully handle additional locations in DiagnosticData.Create#65532
arunchndr merged 1 commit intodotnet:release/dev17.4from
mavasani:PortFix1676229

Conversation

@mavasani
Copy link
Contributor

Ports #65434 to release/dev17.4

Fixes AB#1676229

Handle additional location in different project from the primary location project as well additional location in a document which was subsequently removed from the solution.

Verified that the added unit test fails prior to the product fix.

Fixes [AB#1676229](https://devdiv.visualstudio.com/DefaultCollection/DevDiv/_workitems/edit/1676229)

Handle additional location in different project from the primary location project as well additional location in a document which was subsequently removed from the solution.

Verified that the added unit test fails prior to the product fix.
@arunchndr arunchndr enabled auto-merge November 21, 2022 17:49
@arunchndr
Copy link
Contributor

I cannot imagine how rename tests will be broken by this. Rerunning. At this point, given our infra fixes are landing only in 17.5, we are going to stick to investing in flakiness there, if rerun succeeds

@arunchndr arunchndr merged commit add2a83 into dotnet:release/dev17.4 Nov 21, 2022
@mavasani mavasani deleted the PortFix1676229 branch November 28, 2022 12:18
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.

2 participants