-
Notifications
You must be signed in to change notification settings - Fork 147
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
VSTHRD200 should not trigger on test methods #670
Comments
It is indeed suppressible. This can be done via the msbuild |
Found solution
|
Wait... Could you give a full example of this "Suppress"? This looks interesting so I'd like to know more. EDIT: nevermind, I didn't realize the [*{Controller,StepDefinition,Tests}.cs]
dotnet_diagnostic.VSTHRD200.severity = none
dotnet_analyzer_diagnostic.VSTHRD200.severity = none |
Is your feature request related to a problem? Please describe.
Asynchronous unit test methods don't need Async suffix.
Describe the solution you'd like
Either detect this automatically or add some exclusion rule.
For instance class ends with
Tests
.Describe alternatives you've considered
I do not want to disable it completely in the solution.
It is possible to suppress it in Unit Tests projects but this is not scalable.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: