-
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 be able to have method type configured #1327
Comments
Thanks for this thoughtful write-up. I'd add "local functions" to your list of possible variants. At the moment, I think VSTHRD200 ignores them, and I'm content with that policy. As for the rest of the variations you propose, it adds complexity, and that would need to be accompanied by additional automated testing. We should also consider #1315. |
Is your feature request related to a problem? Please describe.
This analyzer reports all instances of this condition. There are different schools of thought about requiring an Async suffix on awaitable methods.
Describe the solution you'd like
I'd like to see .editorconfig settings to modify the behavior of this analyzer.
dotnet_diagnostic.VSTHRD200.allowed_exceptions = private, internal, protected, no_non_async
Whether it takes this exact form doesn't matter. I just want to shut this warning up for private methods.
Describe alternatives you've considered
I can silence all instances of this analyzer, silence it by file, live with warnings, or follow rules that we don't agree with.
Additional context
I use source generators for binding methods to a view, and the "fix" functionality renames methods but not the generated reference to them in XAML. So this:
which is generated as
would be
DeleteUserAsyncCommand
, and I would have to go through all of my XAML views and fix it.The text was updated successfully, but these errors were encountered: