-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Disable ExtractMethod and VerifyTextSync after 17.9P1 failures #71089
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
Conversation
} | ||
|
||
[IdeFact] | ||
[IdeFact(Skip = "https://github.com/dotnet/roslyn/issues/71088")] |
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.
Note - we don't have to skip these now (they are passing on the current images). But if we don't fix them by the time the queue upgrades, we will need to do this.
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.
Link to queue failure?
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.
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.
Approving - the scouting queue is upgrading tomorrow, so we should get this PR in before that happens
https://dev.azure.com/dnceng/internal/_wiki/wikis/DNCEng%20Services%20Wiki/899/VS2022-Upgrade-Schedule
FYI @sharwell
To be merged if #71088 is not fixed before scouting queue gets promoted to regular queue.