Skip to content

Hot Reload: Fix reloadable type regression#63106

Merged
JoeRobich merged 2 commits intodotnet:release/dev17.3from
tmat:ReloadableTypeCtorFix17.3
Aug 1, 2022
Merged

Hot Reload: Fix reloadable type regression#63106
JoeRobich merged 2 commits intodotnet:release/dev17.3from
tmat:ReloadableTypeCtorFix17.3

Conversation

@tmat
Copy link
Member

@tmat tmat commented Jul 31, 2022

Fixes regression #63103 introduced by #61806

Constructors of the synthesized type C.<>c, which is a nested synthesized type in a reloadable type C were updated instead of added.

VS Feedback: https://dev.azure.com/devdiv/DevDiv/_workitems/edit/1580286
Ask mode bug: https://dev.azure.com/devdiv/DevDiv/_workitems/edit/1581287

@tmat tmat added this to the 17.3 milestone Jul 31, 2022
@tmat tmat requested a review from a team as a code owner July 31, 2022 22:18
@ghost ghost added the Area-Interactive label Jul 31, 2022
@tmat
Copy link
Member Author

tmat commented Jul 31, 2022

@davidwengier PTAL

Copy link
Member

@davidwengier davidwengier left a comment

Choose a reason for hiding this comment

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

Whoops! thanks for fixing :)

@mkArtakMSFT
Copy link

@tmat is this going into 17.3? Which version specifically?

@JoeRobich JoeRobich merged commit 4f0dba1 into dotnet:release/dev17.3 Aug 1, 2022
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.

4 participants