Skip to content

Remove FailFast when rename tracking state is messed up#62955

Merged
sandyarmstrong merged 3 commits intomainfrom
dev/sandy/failslow
Jul 27, 2022
Merged

Remove FailFast when rename tracking state is messed up#62955
sandyarmstrong merged 3 commits intomainfrom
dev/sandy/failslow

Conversation

@sandyarmstrong
Copy link
Copy Markdown
Member

@sandyarmstrong sandyarmstrong commented Jul 26, 2022

We're investigating a crash here in VSMac: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1558256/. We should fix the root cause, but it seems like this should just throw report and return instead of explicitly crashing the process.

@sandyarmstrong sandyarmstrong requested a review from a team as a code owner July 26, 2022 22:21
@ghost ghost added the Area-IDE label Jul 26, 2022
@sandyarmstrong sandyarmstrong changed the title Throw instead of FailFast when rename tracking state is messed up Remove FailFast when rename tracking state is messed up Jul 26, 2022
@sandyarmstrong sandyarmstrong merged commit 1ac1258 into main Jul 27, 2022
@sandyarmstrong sandyarmstrong deleted the dev/sandy/failslow branch July 27, 2022 18:57
@ghost ghost added this to the Next milestone Jul 27, 2022
@dibarbet dibarbet modified the milestones: Next, 17.4 P2 Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants