-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[release/6.0-preview6] Revert "Fix 54025" #54729
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[release/6.0-preview6] Revert "Fix 54025" #54729
Conversation
This reverts commit b2fe667.
|
Tagging subscribers to this area: @tommcdon |
tommcdon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
|
/azp run dotnet-linker-tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
jeffschwMSFT
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved. We should consider this for Preview 6
|
/azp run runtime-dev-innerloop |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@hoyosjs can you double check the failures in CI and then we can merge. |
|
I've retriggered the leg. The issue is a known in bug in XUnit that's starting to cause us more pronounced pain. |
Backport of #54728 to release/6.0-preview6
/cc @hoyosjs
Customer Impact
Reverting as this change breaks several important scenarios in debugging in Linux and macOS.
Testing
Back to heavily tested version - nothing depends on this change other than the bug it tried to fix.
Risk
Very low.