-
Notifications
You must be signed in to change notification settings - Fork 850
Update Microsoft.VisualStudio.Threading.Analyzers to latest #7091
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
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.
Pull request overview
This PR updates the Microsoft.VisualStudio.Threading.Analyzers package from version 17.10.48 to 17.14.15 and addresses a new analyzer warning (VSTHRD002) in the FakeHost.Dispose method by adding a pragma warning suppression and removing the ConfigureAwait(false) call.
Key Changes
- Upgraded Microsoft.VisualStudio.Threading.Analyzers from 17.10.48 to 17.14.15
- Suppressed VSTHRD002 warning for synchronous wait in FakeHost.Dispose method
- Removed ConfigureAwait(false) from the synchronous wait pattern
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| eng/packages/General.props | Updated Microsoft.VisualStudio.Threading.Analyzers package version to 17.14.15 |
| src/Libraries/Microsoft.Extensions.Hosting.Testing/FakeHost.cs | Added VSTHRD002 pragma suppression and removed ConfigureAwait(false) from StopAsync call in Dispose method |
Microsoft Reviewers: Open in CodeFlow